devilhorns pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=d278cd5f49c4677e53d2e224486c4c182a731d95

commit d278cd5f49c4677e53d2e224486c4c182a731d95
Author: Chris Michael <cp.mich...@samsung.com>
Date:   Tue May 12 09:14:26 2015 -0400

    ecore-wayland: Input grab_count is unsigned int, no need for comparison < 0
    
    Summary: As input->grab_count is an unsigned int there is no need for
    the < 0 comparison as that will always return false
    
    @fix
    
    Signed-off-by: Chris Michael <cp.mich...@samsung.com>
---
 src/lib/ecore_wayland/ecore_wl_input.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/lib/ecore_wayland/ecore_wl_input.c 
b/src/lib/ecore_wayland/ecore_wl_input.c
index 294121f..61c32b4 100755
--- a/src/lib/ecore_wayland/ecore_wl_input.c
+++ b/src/lib/ecore_wayland/ecore_wl_input.c
@@ -601,9 +601,8 @@ _ecore_wl_input_cb_pointer_button(void *data, struct 
wl_pointer *pointer EINA_UN
                                         0, button, timestamp);
 
         input->grab_count--;
-        if (input->grab_count < 0)
-            input->grab_count = 0;
-        if ((input->grab) && (input->grab_button == button) && (!state) && 
(!input->grab_count))
+        if ((input->grab) && (input->grab_button == button) && 
+            (!state) && (!input->grab_count))
           ecore_wl_input_ungrab(input);
      }
 
@@ -1110,9 +1109,8 @@ _ecore_wl_input_cb_touch_up(void *data, struct wl_touch 
*touch EINA_UNUSED, unsi
 
    _ecore_wl_input_mouse_up_send(input, input->touch_focus, id, BTN_LEFT, 
timestamp);
    input->grab_count--;
-   if (input->grab_count < 0)
-     input->grab_count = 0;
-   if ((input->grab) && (input->grab_button == BTN_LEFT) && 
(!input->grab_count))
+   if ((input->grab) && (input->grab_button == BTN_LEFT) && 
+       (!input->grab_count))
      ecore_wl_input_ungrab(input);
 }
 

-- 


Reply via email to