[PATCH 1/4] Move region implementation from mi to dix.

Keith Packard keithp at keithp.com
Sat May 22 00:00:40 PDT 2010


On 22 May 2010 03:05:56 +0200, Soeren Sandmann <sandmann at daimi.au.dk> wrote:

> Didn't realize that the "hand-rolled changes" commit used to be part
> of this. The comment I have on that part is that it seems to preserve
> the macro versions of some ops rather than the function versions.

Right, I preserved the implementation that was currently being used
rather than changing anything about how the code works. That seems like
a separate step to me.

> could just call the pixman equivalent like most of the other Region*
> API.

I'd want some performance numbers before/after this change to verify
that nothing 'bad' happens as a result of adding function calls to the
region paths. Nothing I did changed how the code worked aside from
removing some function calls.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100522/9247cfb2/attachment.pgp>


More information about the xorg-devel mailing list