On Fri, 23 Apr 2010 19:52:56 -0700, Jamey Sharp <[email protected]> wrote:
> This fuses two loops, eliminating an intermediate MAXSCREENS-sized
> array.
> 
> Aside from being more efficient, I believe this is equivalent to the
> previous implementation, assuming that
> - each per-screen GC has the graphicsExposures flag set the same way,
>   and

The Xinerama GC ops keep them in sync; check out XineremaCreateGC and friends.

> - the REGION_* macros continue to ignore their screen argument.

Unlikely to ever change now.

I've reviewed this as best I can, but I'd like to see someone actually
give it a try.

Reviewed-by: Keith Packard <[email protected]>

> 
> Signed-off-by: Jamey Sharp <[email protected]>
> ---
> 
> What's up with the REGION_* macros anyway? AFAICT they've never cared
> about their screen argument for as long as we have history in git.

The region functions used to be per-screen.

-- 
[email protected]

Attachment: pgpwmEeFfq7ww.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to