[PATCH xserver] dix: Move {Change, Copy, Destroy}Clip from GCFuncs to Screen

Eric Anholt eric at anholt.net
Wed Mar 1 00:02:01 UTC 2017


Adam Jackson <ajax at redhat.com> writes:

> There are only two behaviors here. Xnest and Xdmx forward the clip list
> to the backing server, other DDXes use mi. But a layer like damage has
> to wrap everything in the GCFuncs vtable even though it really only
> cares about ValidateGC.
>
> Since none of this varies within a screen, we can move the clip
> management funcs up to the ScreenRec and eliminate the empty wrappers.

I tried to come up with a reason I would want a layer to be able to
change the funcs per-gc at runtime, and failed.  Yay for killing
boilerplate.

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


More information about the xorg-devel mailing list