[Issue 19902] hasElaborateCopyConstructor doesn't know about copy constructors

[d-bugmail at puremagic.com]

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

Jonathan M Davis <issues.dlang at jmdavisProg.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |issues.dlang at jmdavisProg.co
                   |                            |m

--- Comment #1 from Jonathan M Davis <issues.dlang at jmdavisProg.com> ---
Indeed. Without it being updated, any existing code that's testing for whether
it's okay to blit a type is broken when dealing with structs that have copy
constructors (e.g. I expect that std.algorithm's swap is currently broken in
such a fashion). I don't know if there's code that would break because it's
really assuming postblit constructors from hasElaborateCopyConstructor rather
than the inability to blit, but I expect that more code will be broken without
updating the trait than with requiring a new one. Also, there's already been at
least one post in D.Learn where someone was confused, because
hasElaborateCopyConstructor wasn't true for types with copy constructors.

--