[PATCH xserver 2/2] xfree86/modes: Eliminate redundant local variable in_range

Michel Dänzer michel at daenzer.net
Fri Dec 25 01:36:34 PST 2015


From: Michel Dänzer <michel.daenzer at amd.com>

Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
---
 hw/xfree86/modes/xf86Cursors.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/hw/xfree86/modes/xf86Cursors.c b/hw/xfree86/modes/xf86Cursors.c
index 27a1b74..5a1dfb1 100644
--- a/hw/xfree86/modes/xf86Cursors.c
+++ b/hw/xfree86/modes/xf86Cursors.c
@@ -395,7 +395,6 @@ xf86_crtc_set_cursor_position(xf86CrtcPtr crtc, int x, int y)
     xf86CursorInfoPtr cursor_info = xf86_config->cursor_info;
     DisplayModePtr mode = &crtc->mode;
     int clip_x, clip_y;
-    Bool in_range;
 
     /*
      * Transform position of cursor on screen
@@ -418,17 +417,15 @@ xf86_crtc_set_cursor_position(xf86CrtcPtr crtc, int x, int y)
     /*
      * Disable the cursor when it is outside the viewport
      */
-    in_range = TRUE;
+    crtc->cursor_in_range = TRUE;
     if (clip_x >= mode->HDisplay || clip_y >= mode->VDisplay ||
         clip_x <= -cursor_info->MaxWidth || clip_y <= -cursor_info->MaxHeight) {
-        in_range = FALSE;
+        crtc->cursor_in_range = FALSE;
         x = 0;
         y = 0;
     }
 
-    crtc->cursor_in_range = in_range;
-
-    if (in_range) {
+    if (crtc->cursor_in_range) {
         crtc->funcs->set_cursor_position(crtc, x, y);
         xf86_crtc_show_cursor(crtc);
     }
-- 
2.6.2



More information about the xorg-devel mailing list