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

List:       kde-commits
Subject:    Re: KDE/kdebase/workspace/plasma/plasma
From:       Matthias Kretz <kretz () kde ! org>
Date:       2008-04-08 10:58:56
Message-ID: 200804081259.01370.kretz () kde ! org
[Download RAW message or body]


On Tuesday 08 April 2008, Andre Duffeck wrote:
> force-sync() the config on application exit until we figure out why it's
> not saved automatically
>
>  PlasmaApp::~PlasmaApp()
>  {
> +    //TODO: This manual sync() should not be necessary. Remove it when
> +    // KConfig was fixed
> +    KGlobal::config()->sync();

Make sure ~KConfig gets called - if it doesn't get called something is leaking 
a reference to the global KConfig object. E.g. KComponentData refs KConfig, 
but there could also be others. The most straightforward way to leak the 
global config is:
new KSharedConfigPtr(KGlobal::config());

-- 
________________________________________________________
Matthias Kretz (Germany)                            <><
http://Vir.homelinux.org/
MatthiasKretz@gmx.net, kretz@kde.org,
Matthias.Kretz@urz.uni-heidelberg.de

["signature.asc" (application/pgp-signature)]

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