[xmonad] Move EWMH support initialization to a startupHook

Daniel Schoepe daniel.schoepe at googlemail.com
Thu Oct 22 08:04:09 EDT 2009


On Sat, Oct 10, 2009 at 11:44:51PM -0600, Justin Bogner wrote:
> This patch will break existing configs.
> 
> EwhmDesktops.setSupported sets some X atoms to constants. This should
> rightfully be done once, in a startup hook, rather than repeatedly in
> the log hook. This complicates the user configs a bit (though they can
> just use Config.Desktop) but fixes problems like setWMName having to be
> called in the log hook if ewmh is being used.
> 

> Sat Oct 10 23:35:38 MDT 2009  Justin Bogner <mail at justinbogner.com>
>   * Move EWMH support initialization to a startupHook
>   
>   Set EWMH support atoms and the window manager name in a startup hook,
>   rather than in the log hook: the log hook occurs far too frequently
>   for it to make sense to set constants with it.

I think that the benefits of this patch outweigh the breakage since
the current behavior is somewhat broken in my opinion.

Applied, thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
Url : http://www.haskell.org/pipermail/xmonad/attachments/20091022/36754f68/attachment.bin


More information about the xmonad mailing list