Re: Planning add-window-matcher extension

[Timo Korvola <Timo Korvola iki fi>]

On Tuesday 24 November 2009 07:43:16 Teika Kazura wrote:
> Hi. In order to make keymap-trans support saner, I'm planning
> an extension for add/remove-window-matcher functions.

Good plan.  The current API is rather bad because you cannot specify 
multiple criteria.

> (add-window-matcher
>   '((WM_NAME "^Terminal$")
>     (WM_CLASS "SuperSpecialUser"))
>       properties... )

You might also consider (define (add-window-matcher rules properties) 
...) rather than (define (add-window-matcher rules . properties) ...).
It is a matter of taste, but I think I would prefer the two-argument 
version because constructions like this would work without messing with 
apply:
(let ((props '((sticky . t) (auto-gravity t))))
  (add-window-matcher '((WM_NAME "^XMMS")) props)
  (add-window-matcher '((WM_CLASS "^Audacious/")) props))

-- 
	Timo Korvola		<URL:http://www.iki.fi/tkorvola>