diff options
| author | Peter Hutterer <peter.hutterer@who-t.net> | 2012-08-23 15:00:24 +1000 |
|---|---|---|
| committer | Peter Hutterer <peter.hutterer@who-t.net> | 2012-10-04 13:24:44 +1000 |
| commit | f64254d85e731d0b4369d871a9a735b03f283ba6 (patch) | |
| tree | aa2b0e9ad07d0a9bc0ef29d2eb01e509bf3f8bef /mi | |
| parent | 314776eb369ca2e438907795ae030dd743c281fc (diff) | |
mi: drop two useless conditions in miPointerSetPosition
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.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Keith Packard <keithp@keithp.com>
Diffstat (limited to 'mi')
| -rw-r--r-- | mi/mipointer.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/mi/mipointer.c b/mi/mipointer.c index 4defaf5ec..f34506326 100644 --- a/mi/mipointer.c +++ b/mi/mipointer.c @@ -575,13 +575,8 @@ miPointerSetPosition(DeviceIntPtr pDev, int mode, double *screenx, miPointerPtr pPointer; - if (!pDev) - return NULL; - pPointer = MIPOINTER(pDev); pScreen = pPointer->pScreen; - if (!pScreen) - return NULL; /* called before ready */ x = trunc(*screenx); y = trunc(*screeny); |