[Libreoffice] [PATCH] Remove unused code

Eike Rathke erack at redhat.com
Fri Jan 20 16:34:06 PST 2012

Hi Michael,

On Friday, 2012-01-20 21:57:25 +0100, Michael Stahl wrote:

> > -VclEventListeners2::~VclEventListeners2()
> > -{
> > -}
> i'm not sure this is right; if a destructor of a class is unused, but
> its other methods are used, then that usually indicates a memory leak:
> the destructor is never called because all instances are leaked.

Yup.. given that we have

class VclEventListeners2 : public vcl::DeletionNotifier

maybe it's sufficient to make the dtor virtual if there's a delete
baseclasspointer somewhere?


LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3  9E96 2F1A D073 293C 05FD
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120121/90f603da/attachment.pgp>

More information about the LibreOffice mailing list