Re: [dev] Re: [PATCH] wmii - client mapping events

On Mon, Mar 28, 2011 at 03:53:26PM -0700, Suraj Kurapati wrote:
>On Sun, Mar 27, 2011 at 1:04 AM, Suraj Kurapati <sunaku_AT_gmail.com> wrote:
>> The following patch against wmii-hg r2785 allows my wmiirc to know
>> when clients are added to or removed from the current view.
>
>Here is an updated patch that renames the events to ClientAttach and
>ClientDetach and, more importantly, fires those events only when the
>current view is actually changed. No need for special logic to detect
>valid boundaries of MapClient and UnmapClient events. Cheers.

I like this idea, but I don't like that it only applies to the
selected view. I'll add an attach event that includes the client
and the view. I'm not sure I see the point of the map and unmap
events.

-- 
Kris Maglione
There are two ways of constructing a software design.  One way is to
make it so simple that there are obviously no deficiencies.  And the
other way is to make it so complicated that there are no obvious
deficiencies.
	--C.A.R. Hoare