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@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, fa...@kde.org, sponsored by Qt Software @ Nokia to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to