[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