[PATCH xserver 3/4] animcur: Run the timer from the device, not the screen
Keith Packard
keithp at keithp.com
Mon Nov 13 01:02:06 UTC 2017
Adam Jackson <ajax at redhat.com> writes:
> + /*
> + * Not a simple Unwrap/Wrap as this isn't called along the DisplayCursor
> + * wrapper chain.
> + */
> + pScreen->DisplayCursor = as->DisplayCursor;
> + (void) (*pScreen->DisplayCursor) (dev, pScreen, ac->elts[elt].pCursor);
> + as->DisplayCursor = pScreen->DisplayCursor;
> + pScreen->DisplayCursor = DisplayCursor;
I'm probably reading this out of context, but if this isn't called along
the usual DisplayCursor wrapper chain, then it's presumptuous to smash
in our DisplayCursor pointer; shouldn't it preserve the previous value
and restore that instead?
--
-keith
-------------- 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/20171112/baaa2298/attachment.sig>
More information about the xorg-devel
mailing list