From 53d617276e45da5617e6f49138154bc4f2f7e97d Mon Sep 17 00:00:00 2001 From: Vincent Torri Date: Mon, 27 May 2019 15:56:41 +0900 Subject: [PATCH] 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 34c9030..2e007cc 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; -- 2.7.4