On Wed, 2009-07-22 at 21:42 +0200, Maarten Maathuis wrote:
> - Rename exa_migration.c to exa_migration_classic.c
> - Create a few seperate functions and a few private function pointers.
> - Replace a few if conditions with a check for pExaPix->pDamage instead.
> - This is in preperation of a third scheme that lies somewhere in between.

Some things like the file rename or the move of struct _ExaMigrationRec
seem gratuitous, but otherwise it looks okay. However, I don't have a
very good idea yet where you're going with the third scheme, might be
easier to review this patch together with that.


-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer
_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to