On Fri, May 21 2010, Daniel Graña wrote:

> Here is another patch that unregister systray if no systray widget is found
> for a wibox.
> see http://gist.github.com/409310

Hum, that does not seems good to me, and actually, that makes me wonder
if the first patch is.

I think you register/unregister at the wrong time. You should register
when the first systray widget is created, and unregister when the last
systray widget is unregistered.

In your approach, the X systray is not registered if the systray widget
is never attached to a wibox or displayed. While this may works in
common usage, it's not valid in theory.

-- 
Julien Danjou
// ᐰ <[email protected]>   http://julien.danjou.info

Attachment: pgpJj1Ikh79EQ.pgp
Description: PGP signature

Reply via email to