.spec: add and -Werror

Change-Id: I9528dd92153a5d052dd242c0b09812df01969526

diff --git a/packaging/e-mod-tizen-gesture.spec b/packaging/e-mod-tizen-gesture.spec
index f21d5f25e338d58c6bcda90f587db6ac5c241954..74e787e4bdf2636f084d6b6c9cae8475aed8a7f0 100644 (file)
--- a/packaging/e-mod-tizen-gesture.spec
+++ b/packaging/e-mod-tizen-gesture.spec
@@ -26,7 +26,7 @@ This package is a the Enlightenment Gesture Module for Tizen.
 %build
 
 export GC_SECTIONS_FLAGS="-fdata-sections -ffunction-sections -Wl,--gc-sections"
-export CFLAGS+=" -Wall -g -fPIC -rdynamic ${GC_SECTIONS_FLAGS} -DE_LOGGING=1 "
+export CFLAGS+=" -Wall -Werror -g -fPIC -rdynamic ${GC_SECTIONS_FLAGS} -DE_LOGGING=1 "
 export LDFLAGS+=" -Wl,--hash-style=both -Wl,--as-needed -Wl,--rpath=/usr/lib"
 
 %autogen

diff --git a/src/e_mod_main.h b/src/e_mod_main.h
index 61d4e4932fe6ec698be2c1baf444c68c92ca70c4..c3cd7f1f36c12ed286db172f3554fe634ef74c66 100644 (file)
--- a/src/e_mod_main.h
+++ b/src/e_mod_main.h
@@ -55,7 +55,9 @@
 
 #define E_GESTURE_PINCH_MOVING_DISTANCE_RANGE 15.0
 
+#ifndef ABS
 #define ABS(x) (((x)>0)?(x):-(x))
+#endif
 #define RAD2DEG(x) ((x) * 57.295779513)
 
 typedef struct _E_Gesture E_Gesture;
@@ -111,7 +113,7 @@ enum _E_Gesture_Tap_State
    E_GESTURE_TAP_STATE_NONE,
    E_GESTURE_TAP_STATE_READY, // tap is required, idle
    E_GESTURE_TAP_STATE_START, // first finger is pressed
-   E_GESTURE_TAP_STATE_PROCESS, // all fingers are pressed or first release
+   E_GESTURE_TAP_STATE_PROCESS, // tap.time_start has passed and all fingers are pressed or first release
    E_GESTURE_TAP_STATE_WAIT, // all fingers are released and wait next tap
    E_GESTURE_TAP_STATE_DONE
 };
@@ -119,9 +121,9 @@ enum _E_Gesture_Tap_State
 enum _E_Gesture_PanPinch_State
 {
    E_GESTURE_PANPINCH_STATE_NONE,
-   E_GESTURE_PANPINCH_STATE_READY,
-   E_GESTURE_PANPINCH_STATE_START,
-   E_GESTURE_PANPINCH_STATE_MOVING,
+   E_GESTURE_PANPINCH_STATE_READY, // first finger is pressed
+   E_GESTURE_PANPINCH_STATE_START, // pan.time_start passed and all fingers are pressed
+   E_GESTURE_PANPINCH_STATE_MOVING, // all fingers are moving
    E_GESTURE_PANPINCH_STATE_DONE
 };