Kcontrol memory leak (patches included)

Waldo Bastian bastian at kde.org
Wed Jul 14 20:46:44 BST 2004


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wed July 14 2004 20:17, Albert Astals Cid wrote:
> Ok then i attach two patches.
>
> KCMdiff is a patch to kcmodule.cpp and .h to add the functions Frans
> suggested.
> artsKCMdiff is a patch to port arts kcontrol module to the new functions.
>
> I think they are fine, but for some strange reason i don't get the about
> option all i get in kcontrol Help is a disabled entry saying "About Current
> Module".
>
> Can someone please test the patches and tell me if they see the
> personalitzed about in arts module or they get "About Current Module" in
> grey ? (i think my kde stable version may be interfering)

KAboutData is doesn't have a proper copy constructor (unless it was recently 
added) so new KAboutData( aboutData ) doesn't exactly work. Why not change 
the function from setAboutData(const KAboutData&) to setAboutData(KAboutData 
*)?

Cheers,
Waldo
- -- 
bastian at kde.org  |   KDE Community World Summit 2004  |  bastian at suse.com
bastian at kde.org  | 21-29 August, Ludwigsburg, Germany |  bastian at suse.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)

iD8DBQFA9Y2kN4pvrENfboIRAjWgAJ4+5gnZjL6Cfti9l7J6dExAyzG3fQCfTPeY
2y4fr7ug9vYgOTM+J5LDfHM=
=FO7S
-----END PGP SIGNATURE-----




More information about the kde-core-devel mailing list