[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: KDE/kdebase/workspace/plasma/applets/kickoff/core
From:       David Faure <faure () kde ! org>
Date:       2009-07-20 13:33:06
Message-ID: 200907201533.07004.faure () kde ! org
[Download RAW message or body]

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


[prev in list] [next in list] [prev in thread] [next in thread]