disabling loading of kded autoload modules if KDE desktop is not running

Waldo Bastian bastian at kde.org
Thu May 13 13:10:59 BST 2004


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

On Thu May 13 2004 13:31, Lubos Lunak wrote:
>  The attached patch changes kded so that autoload modules are loaded on
> kded startup only if that's also KDE startup. Can anybody see a problem
> with that? I do't think there's any reason why just starting e.g. KWrite
> outside of KDE should also load kwrited, klaptopdaemon, kmilo and what not.
> In case some of the explicitly CC-ed people (that's people responsible for
> various autoload=true according to cvs annotate) have a good reason why not
> to apply this patch, please speak up.

Suggest to change:

bool kde_running = !( getenv( "KDE_FULL_SESSION" ) == NULL || 
getenv( "KDE_FULL_SESSION" )[ 0 ] == '\0' );
 
to something like:

bool kde_running = ! QCString(getenv( "KDE_FULL_SESSION" )).isEmpty();

Cheers,
Waldo
- -- 
bastian at kde.org  |   Novell BrainShare Europe 2004   |  bastian at suse.com
bastian at kde.org  | 12-18 September, Barcelona, Spain |  bastian at suse.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)

iD8DBQFAo2XTN4pvrENfboIRAoJqAJ0fa/KKEs6ZWLyNSRGZluaSJGt3hQCgjsS3
ur1RrqeZ6FNFjl7q3qRlOK4=
=5flt
-----END PGP SIGNATURE-----




More information about the kde-core-devel mailing list