Review Request: fix a memory leak in ktip

Michael Pyne mpyne at
Fri Jul 24 22:34:55 BST 2009

On Friday 24 July 2009 15:22:22 Chani wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> -----------------------------------------------------------
> Ship it!
> /trunk/KDE/kdelibs/kdeui/dialogs/ktip.cpp
> <>
>     it's called Private, it's defined in the cpp file not a .h - it's
> private API. :) that means you don't have to worry about BC requirements.
> I'm not actually sure whether adding a destructor is BC...

I *think* it is BC as if the destructor is virtual then it should already have 
an entry in the vtable, and if the destructor is non-virtual then it's not a 

Of course it doesn't matter here anyways.

 - Michael Pyne
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <>

More information about the kde-core-devel mailing list