[Nepomuk] [nepomuk-kde] serviceOwnerChanged usage

[Sebastian Trüg]

Hi Aaron,

(for the future please use the new nepomuk mailing list hosted at kde:
https://mail.kde.org/mailman/listinfo/nepomuk)

I am aware of the necessary changes. And I would be happy, too if "one
of the nepomukians will be willing". ;)
In the end it will probably be me... I suppose I need tutoring when it
comes to delegation of work... any takers? :P

Cheers,
Sebastian

On 03/26/2010 05:58 AM, Aaron J. Seigo wrote:
> hi all...
>
> i've spent parts of the last few days replacing usage of 
> QDBusConnection::interface()'s serviceOwnerChanged() signal, which fires 
> whenever any service appears or disappears on the bus, with 
> QDBusServiceWatcher which is far more targetted. this lets processes sleep 
> much more; seeing as this often comes into play when starting applications 
> it's pretty important.
>
> i've been selfish, however: i've concentrated on fixing things that plasma-
> desktop uses directly! yes, i'm a bastard.
>
> i'm writing this list because nepomuk code (ab)uses the serviceOwnerChanged 
> signal in several places. i'm hoping one of the nepomukians will be willing to 
> move to QDBusServiceWatcher.
>
> here are the places that need work in kdelibs:
>
> nepomuk/core/resourcemanager.cpp
>
> and in kdebase/runtime:
>
> nepomuk/kcm/nepomukserverkcm.cpp:             SIGNAL( serviceOwnerChanged( 
> const QString&, const QString&, const QString& ) ),
> nepomuk/server/servicecontroller.cpp:                 SIGNAL( 
> serviceOwnerChanged( const QString&, const QString&, const QString& ) ),
> nepomuk/kioslaves/search/kdedmodule/nepomuksearchmodule.cpp:    // connect to 
> serviceOwnerChanged to catch crashed clients that never unregistered
> nepomuk/kioslaves/search/kdedmodule/nepomuksearchmodule.cpp:             
> SIGNAL( serviceOwnerChanged( const QString&, const QString&, const QString& ) 
> ),
> nepomuk/services/queryservice/queryservice.cpp:             SIGNAL( 
> serviceOwnerChanged( const QString&, const QString&, const QString& ) ),
>
> :)
>
>