On Mon, 6 Dec 2010 14:59:17 -0800, James Jones <jajo...@nvidia.com> wrote:

> Took me a while to re-verify, but it's reworked and ready for review again.  
> Thanks for the feedback.  Note it won't help a lot with bisect because XSync 
> wasn't using protocol-versions.h, but it is better organized now.  I avoided 
> squashing too many commits because I didn't want to make the individual 
> patches too large, or change and move the same code in one commit (kills diff 
> readability).

The patch series was quite readable, and there's not much to be done
about the broken protocol-versions.h stuff. Thanks for getting that
applied first.

I think this is ready for merging; the only potential change would be to
use a union type instead of casts for all of the subtyping with sync
objects  vs counters/fences. We should be able to clean that up later
though, if you think it's a reasonable plan.

-- 
keith.pack...@intel.com

Attachment: pgpsgpCOcDQrq.pgp
Description: PGP signature

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to