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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |petar.p.kirov@gmail.com

--- Comment #1 from ZombineDev <petar.p.kirov@gmail.com> ---
I guess what you meant to say is that on Linux the workaround is:
void dtor1() { }
void dtor2() { }

--

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

Manu <turkeyman@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |turkeyman@gmail.com

--- Comment #2 from Manu <turkeyman@gmail.com> ---
Just to be clear, this is only for classes with a virtual destructor in the
base?
In the case where linux needs 2 destructors, what are they each for? How should
they each be defined?

--

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

--- Comment #3 from Manu <turkeyman@gmail.com> ---
I'm noticing a problem with constructors. Is this related?

'extern(C++) this() {}' doesn't seem to emit a C++ constructor, it emit's a
call to Class::__ctor, which isn't what C++ calls constructors.
Do I need to duplicate constructors on the D side?

--

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

--- Comment #4 from github-bugzilla@puremagic.com ---
Commit pushed to master at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/41151beba9fdd3a649729f2fae1ef51fd71c725e Fix issue 15589 - extern(C++) virtual destructors are not put in vtbl[]

--

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

github-bugzilla@puremagic.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--

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

--- Comment #5 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/ab0a7bf76dbf0a02e826a1a9b54f9d813c7a114a fix issue 15589 - cleanup after PR #8277

fix mangling, add shim functions for vtbl and type info, fix dtor inside Attribdeclaration, use Id-pool

https://github.com/dlang/dmd/commit/d08e0fb3bff6767bd28516f53393083d86f63045 Merge pull request #8349 from rainers/issue15589

fix issue 15589 - cleanup/improve PR #8277 - C++ dtors in vtbl merged-on-behalf-of: Walter Bright <WalterBright@users.noreply.github.com>

--