[PATCH] Replace miPointerCurrentScreen with miPointerGetScreen.
Daniel Stone
daniel at fooishbar.org
Sat Oct 3 18:03:14 PDT 2009
Hi,
On Fri, Oct 02, 2009 at 02:54:57PM -0700, Jamey Sharp wrote:
> I get the impression from glancing at the code that you're right, but
> xf86RandRSetConfig begins with
>
> miPointerGetPosition(inputInfo.pointer, &px, &py);
>
> and this if block ends with
>
> (*pScreen->SetCursorPosition) (inputInfo.pointer, pScreen, ...
>
> so there's quite a bit more involved in such a change.
>
> Meanwhile, this simple patch at least makes this call consistent with
> the two near it, and fixes the compiler warning.
To be honest, I'd rather keep the warning in this case, so we know that
the code needs to be fixed.
> I suspect most of this code could go away anyway, in a world where
> people rarely use viewport panning. I'd guess the real problem is that
> xf86SwitchMode goes to a bunch of trouble to move all the pointers
> around and then this function feebly attempts to put one of them back.
The panning code is new. :\
Cheers,
Daniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-devel/attachments/20091004/5ba6357c/attachment.pgp
More information about the xorg-devel
mailing list