Adam Jackson <a...@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
signature.asc
Description: PGP signature
_______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel