[PATCH synaptics 05/21] Use unsigned long for historical time, not int

Daniel Stone daniel at fooishbar.org
Tue Jun 14 10:05:53 PDT 2011


Since we get time in unsigned longs rather than int, truncating just
seems a bit silly.

Signed-off-by: Daniel Stone <daniel at fooishbar.org>
Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>
---
 src/synaptics.c    |    2 +-
 src/synapticsstr.h |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/synaptics.c b/src/synaptics.c
index 40eca99..2f6a8b9 100644
--- a/src/synaptics.c
+++ b/src/synaptics.c
@@ -1693,7 +1693,7 @@ HandleTapProcessing(SynapticsPrivate *priv, struct SynapticsHwState *hw,
 #define HIST(a) (priv->move_hist[((priv->hist_index - (a) + SYNAPTICS_MOVE_HISTORY) % SYNAPTICS_MOVE_HISTORY)])
 
 static void
-store_history(SynapticsPrivate *priv, int x, int y, unsigned int millis)
+store_history(SynapticsPrivate *priv, int x, int y, unsigned long millis)
 {
     int idx = (priv->hist_index + 1) % SYNAPTICS_MOVE_HISTORY;
     priv->move_hist[idx].x = x;
diff --git a/src/synapticsstr.h b/src/synapticsstr.h
index b2fecbe..28bb03e 100644
--- a/src/synapticsstr.h
+++ b/src/synapticsstr.h
@@ -48,7 +48,7 @@
 typedef struct _SynapticsMoveHist
 {
     int x, y;
-    int millis;
+    unsigned long millis;
 } SynapticsMoveHistRec;
 
 enum FingerState {		/* Note! The order matters. Compared with < operator. */
-- 
1.7.5.3



More information about the xorg-devel mailing list