How invasive is multihead patch?

[Janek Kozicki <janek_listy wp pl>]

Andrea Vettorello said:     (by the date of Thu, 17 Jan 2008 09:57:27 +0100)
> On Jan 17, 2008 1:17 AM, Janek Kozicki <janek_listy wp pl> wrote:
> >   http://sawfish.wikia.com/wiki/Alpha_backgrounds ?
> Maybe GDK/GTK alpha handling was fixed in the meantime?


Hello,

thanks for your comment.

Now I need opinion about the multihead patch, please see:

  http://mail.gnome.org/archives/sawfish-list/2006-August/msg00007.html
  http://sawfish.wikia.com/wiki/Multihead_placed_patch

My problem is that I am in no position to judge how bad or invasive
this patch is. Or maybe it is plainly fixing a coding mistake (which
I can't recognize here)? 

I have dualhead also, and I can reproduce the bad behaviour described
in there. So I can imagine how annoying it can be for someone who
uses this placement method (centered/centered-on-parent).

So this is why I'm a bit inclied to apply this patch. But there is no
feedback from you on this matter. And I don't want to spoil sawfish out
of my ignorance (just because I like dualhead setups) so I need your
opinion.

Even if you don't have dualhead - look at the code: Is it dirty or not?

Personally I use first-fit/under-pointer so this problem has never
affected me.

thanks
-- 
Janek Kozicki                                                         |