KDE/kdebase/workspace/plasma/applets/kickoff/core

[David Faure]

On Saturday 11 July 2009, Michael Jansen wrote:
> SVN commit 994994 by mjansen:
> 
> Fix runtime warning:
> 
>     type=Qt::AutoConnection) at /home/mjansen/kde/trunk/src/qtmaster/src/corelib/kernel/qobject.cpp:2534
>     at /home/mjansen/kde/trunk/src/kdebase/workspace/plasma/applets/kickoff/core/applicationmodel.cpp:280
> 
> CCMAIL: plasma-devel at kde.org
> 
>  M  +1 -1      applicationmodel.cpp  
> 
> 
> --- trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/applicationmodel.cpp #994993:994994
> @@ -277,7 +277,7 @@
>      (void)new KickoffAdaptor(this);
>      QDBusConnection::sessionBus().registerObject("/kickoff", this);
>      dbus.connect(QString(), "/kickoff", "org.kde.plasma", "reloadMenu", this, SLOT(reloadMenu()));
> -    connect(KSycoca::self(), SIGNAL(databaseChanged()), this, SLOT(checkSycocaChange()));
> +    connect(KSycoca::self(), SIGNAL(databaseChanged(const QStringList&)), this, SLOT(checkSycocaChange()));

It would be a good idea to check that the stringlist includes apps or xdgdata-apps
and bailing out early if not. That's the whole point of the argument: to not update
apps stuff if e.g. only mimetypes or servicetypes changed.

-- 
David Faure, faure at kde.org, sponsored by Qt Software @ Nokia to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).