[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Kcontrol memory leak (patches included)
From:       Waldo Bastian <bastian () kde ! org>
Date:       2004-07-14 19:46:44
Message-ID: 200407142146.44807.bastian () kde ! org
[Download RAW message or body]

-----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@kde.org  |   KDE Community World Summit 2004  |  bastian@suse.com
bastian@kde.org  | 21-29 August, Ludwigsburg, Germany |  bastian@suse.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)

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

[prev in list] [next in list] [prev in thread] [next in thread]