[EGIT] [core/efl] master 01/01: Evas gesture: do not use abs when dealing with double, use fabs instead

[Vincent Torri]

hermet pushed a commit to branch master.

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

commit 0732606118ed5d740c049b173e60c02c550c34d5
Author: Vincent Torri <vincent.to...@gmail.com>
Date:   Mon May 27 15:56:41 2019 +0900

    Evas gesture: do not use abs when dealing with double, use fabs instead
    
    Summary: clang reports a warning here. We compare the result of abs 
(casting to an int, returning an int) to a double value. Hence we should use 
fabs instead
    
    Test Plan: compilation
    
    Reviewers: Jaehyun, Hermet
    
    Reviewed By: Hermet
    
    Subscribers: cedric, #reviewers, #committers
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D9024
---
 src/lib/evas/gesture/efl_canvas_gesture_recognizer_momentum.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib/evas/gesture/efl_canvas_gesture_recognizer_momentum.c 
b/src/lib/evas/gesture/efl_canvas_gesture_recognizer_momentum.c
index 34c9030ad3..2e007cc0f9 100644
--- a/src/lib/evas/gesture/efl_canvas_gesture_recognizer_momentum.c
+++ b/src/lib/evas/gesture/efl_canvas_gesture_recognizer_momentum.c
@@ -175,8 +175,8 @@ 
_efl_canvas_gesture_recognizer_momentum_efl_canvas_gesture_recognizer_recognize(
                   pd->t_end = efl_gesture_touch_cur_timestamp_get(event);
            efl_gesture_hotspot_set(gesture, pd->end_line);
 
-           if ((abs(md->momentum.x) > EFL_GESTURE_MINIMUM_MOMENTUM) ||
-               (abs(md->momentum.y) > EFL_GESTURE_MINIMUM_MOMENTUM))
+           if ((fabs(md->momentum.x) > EFL_GESTURE_MINIMUM_MOMENTUM) ||
+               (fabs(md->momentum.y) > EFL_GESTURE_MINIMUM_MOMENTUM))
                 result = EFL_GESTURE_FINISH;
            else
                 result = EFL_GESTURE_CANCEL;

--