[patch] add option for disabling kwinstartmenu
Ralf Habacker
ralf.habacker at freenet.de
Wed Apr 16 08:22:01 CEST 2008
Jarosław Staniek schrieb:
> for review,
>
+ KConfigGroup group( &d->config, "General" );
+ if (!group.readEntry("Enabled", true))
+ return;
+
returning immediatly in the constructor disables the dbus interface which is initiated later
Would it not be better in case of menu creating disabled not to call the following connect (and disconnect on destruction) ?
connect(d->ksycoca, SIGNAL(databaseChanged()), this, SLOT(databaseChanged()));
Ralf
More information about the Kde-windows
mailing list