On Wed, Sep 28, 2011 at 11:02:42AM -0700, Aaron Plattner wrote: > Xserer-spec.xml still lists ChangeClip in the "mi" section in the > list of server routines for porting, and DestroyClip in the "ddx" > section.
Good catch! Amended locally and on my published review-pending branch, thanks. > Can't you drop the CT_PIXMAP case in xnestChangeClip? I guess it > doesn't actually matter since it's just dead code and it gets > removed in patch 2/3 anyway... Yeah, I didn't want to make the assumption that CT_PIXMAP wasn't used until I could first assume that only the mi implementation of clip management needed review, so I think the series stands better in this order. > Other than that, > Reviewed-by: Aaron Plattner <[email protected]> Thank you! This patch and my fix to make kdrive build again are now on my "reviewed" branch, ready for Keith to pull, though I'll wait a bit to see if you're reviewing the other two patches. :-) Jamey
signature.asc
Description: Digital signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
