removing dependencies from kactivitymanagerd

Martin Gräßlin mgraesslin at kde.org
Tue Oct 4 15:17:52 UTC 2011


On Tuesday 04 October 2011 16:33:03 Aaron J. Seigo wrote:
> hi all...
> 
> i experimented a bit this morning with cutting the fat from kactivitymanagerd. 
> in particular i focussed on the following.
> 
> KUniqueApplication: this lives in kdeui ... just to provide a way to have only 
> one instance of the app. ugh. in Frameworks there is libkdbus which has a 
> KDBusService which provides the same capabilities. porting to that proved 
> quite simple and straightforward and let ActivityManager become a 
> QCoreApplication subclass.
> 
> KWindowSystem: this one is more difficult. it is used only to track the 
> comings and goings of windows. no other features are used. and it ends up 
> causing a QWidget to be created, which KWindowSystem uses to filter x11 
> events. i don't have a great solution for this one .. but it would be very 
> nice to have something that doesn't pull in such a heavy set of dependencies 
> just to watch window states. i don't know if this would be general-purpose 
> enough to end up in Frameworks (my gut says "no") but writing a simple class 
> that can have a window-system-specific implementation that alerts when the 
> window focus changes and windows are closed would make a lot of sense for 
> kactivitymanagerd.
> 
> so, i have a patch for the former, but nothing for the latter. any takers? :)
Well for Wayland we need to change KWindowSystem anyway and I don't want to low level protocols. So who wants to 
design a D-Bus interface for broadcasting such methods and add a nice wrapper around it? Adding support for it in KWin 
should be simple. What do you think?

Cheers
Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111004/66a3d75a/attachment.sig>


More information about the Plasma-devel mailing list