[PATCH xserver 3/4] animcur: Run the timer from the device, not the screen
Adam Jackson
ajax at redhat.com
Tue Nov 14 15:11:35 UTC 2017
On Sun, 2017-11-12 at 17:02 -0800, Keith Packard wrote:
> 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?
You added that comment when you added XFIXES to XFree86, presumably you
know what it means better than I do.
- ajax
More information about the xorg-devel
mailing list