https://issues.dlang.org/show_bug.cgi?id=15829

ZombineDev <petar.p.kirov@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #1 from ZombineDev <petar.p.kirov@gmail.com> ---
PR: https://github.com/D-Programming-Language/phobos/pull/4119

--

https://issues.dlang.org/show_bug.cgi?id=15829

Nick Treleaven <ntrel-pub@mybtinternet.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ntrel-pub@mybtinternet.com

--- Comment #2 from Nick Treleaven <ntrel-pub@mybtinternet.com> ---
Just for the record, this is by design according to the docs.

It would be nice if hasElaborateDestructor didn't compile for reference types. I don't know if the breakage is acceptable, but at least it would enforce correct usage.

--

https://issues.dlang.org/show_bug.cgi?id=15829

--- Comment #3 from ZombineDev <petar.p.kirov@gmail.com> ---
But what about generic code that already (wrongly) relies on hasElaborateDestructor (such as std.experimental.allocator)? Should we add hasElaborateDestructor2 that handles both classes and structs and errors on unions?

--

https://issues.dlang.org/show_bug.cgi?id=15829

b2.temp@gmx.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |b2.temp@gmx.com
         Resolution|---                         |WONTFIX

--- Comment #4 from b2.temp@gmx.com ---
No to accept this was justified here:

https://github.com/dlang/phobos/pull/4119#issuecomment-226930708

--

https://issues.dlang.org/show_bug.cgi?id=15829

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|b2.temp@gmx.com             |

--