[compiz] [PATCH] Notification on window restacking

[Erkin Bahceci]

On 3/24/07, Mike Dransfield <mike at blueroot.co.uk> wrote:
> Erkin Bahceci wrote:
> > This patch adds notification on window restacking. Plugins may need to
> > do something when that happens. For example, it is used in the new
> > version of animation plugin to perform a fade effect when a window
> > moves above or below other windows (in the plugin's settings this
> > effect is selected as a focus effect, but it's actually a restack
> > effect).
>
> Are you not able to listen for restacking client messages,
> or changes to _NET_CLIENT_LIST_STACKING ?
>

Ok, but that way don't we need to store a list of windows (current
order) in the plugin and compare the list with the new list when
_NET_CLIENT_LIST_STACKING changes just to find the window that has
been restacked (like updateClientListForScreen() does)? (Is there an
easier way to find that window with the method you suggest?) The
restacked window is already identified in restackWindow(). Wouldn't it
be more efficient to use this already available information and send a
notification from there instead of doing an extra list comparison in
each plugin that needs that notification?

Regards,
Erkin