From kde-panel-devel  Mon Jul 20 13:33:06 2009
From: David Faure <faure () kde ! org>
Date: Mon, 20 Jul 2009 13:33:06 +0000
To: kde-panel-devel
Subject: Re: KDE/kdebase/workspace/plasma/applets/kickoff/core
Message-Id: <200907201533.07004.faure () kde ! org>
X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=124809685606672

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, faure@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