[PATCH 0/5] Document screen proc wrapping and fix places where it was wrong

Eric Anholt eric at anholt.net
Mon Jun 16 01:14:14 PDT 2014


Keith Packard <keithp at keithp.com> writes:

> Here's the patches I proposed which split the fixing of the block
> handler wrapping into five individual fixes.
>
>  [PATCH 1/5] Document how to correctly wrap screen procedures
>
> Adds the giant comment to scrnintstr.h
>
>  [PATCH 2/5] hw/xfree86: Fix block handler wrapping in xf86Rotate
>
> Fixes the demonstrated bug
>
>  [PATCH 3/5] mi: Fix block handler wrapping in miSprite
>
> Fixes incorrect wrapping of the BlockHandler in miSprite. Note that
> all other wrapping in misprite was already correct...
>
>  [PATCH 4/5] hw/xfree86: Fix VGA arbiter screen proc wrapping
>
> Fixes all of the screen wrappers in the VGA arbiter.
>
>  [PATCH 5/5] hw/xfree86: Let xf86Rotate leave the BlockHandler
>
> Finally, removes the rotate block handler when possible.

Sorry, I misplaced this series.  It looks great.

Reviewed-by: Eric Anholt <eric at anholt.net>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140616/c44f46b8/attachment.sig>


More information about the xorg-devel mailing list