[PATCH] mi: drop two useless conditions in miPointerSetPosition

Keith Packard keithp at keithp.com
Thu Aug 23 10:03:42 PDT 2012


Peter Hutterer <peter.hutterer at who-t.net> writes:

> pDev cannot be NULL here since fill_pointer_events is the only caller.
> And if the screen is NULL, then the device tries to send events before it is
> fully initialised. That certainly shouldn't happen and would be a bug
> elsewhere.

Looks good. This should be queued for 1.14 though; it's not a bug fix.

Reviewed-by: Keith Packard <keithp at keithp.com>

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120823/eadf63f8/attachment-0001.pgp>


More information about the xorg-devel mailing list