janitor: Simplify gesture tracing

It is more convenient to print out the gesture at will than
having trace commands all over the code.

Signed-off-by: Henrik Rydberg <rydberg@euromail.se>

1 files changed, 16 insertions, 35 deletions

diff --git a/driver/multitouch.c b/driver/multitouch.c
index 7233aa2..0809948 100644
--- a/driver/multitouch.c
+++ b/driver/multitouch.c
@@ -34,15 +34,6 @@ static const float hswipe_fraction = 0.25;
 static const float scale_fraction = 0.05;
 static const float rot_fraction = 0.05;
 
-/* flip these to enable event debugging */
-#if 1
-#define TRACE1(format, arg1)
-#define TRACE2(format, arg1, arg2)
-#else
-#define TRACE1(format, arg1) xf86Msg(X_INFO, format, arg1)
-#define TRACE2(format, arg1, arg2) xf86Msg(X_INFO, format, arg1, arg2)
-#endif
-
 /* button mapping simplified */
 #define PROPMAP(m, x, y) m[x] = XIGetKnownProperty(y)
 
@@ -217,12 +208,6 @@ static void handle_gestures(LocalDevicePtr local,
                             const struct Capabilities *caps)
 {
         static int vscroll, hscroll, vswipe, hswipe, scale, rot;
-        int vscrollstep = 1 + vscroll_fraction * get_cap_ysize(caps);
-        int hscrollstep = 1 + hscroll_fraction * get_cap_xsize(caps);
-        int vswipestep = 1 + vswipe_fraction * get_cap_ysize(caps);
-        int hswipestep = 1 + hswipe_fraction * get_cap_xsize(caps);
-        int scalestep = 1 + scale_fraction * get_cap_xsize(caps);
-        int rotstep = 1 + rot_fraction * get_cap_xsize(caps);
         int i;
         if (!gs->same_fingers) {
                 vscroll = 0;
@@ -231,40 +216,36 @@ static void handle_gestures(LocalDevicePtr local,
                 hswipe = 0;
         }
         for (i = 0; i < DIM_BUTTON; i++) {
-                if (GETBIT(gs->btmask, i)) {
+                if (GETBIT(gs->btmask, i))
                         xf86PostButtonEvent(local->dev, FALSE,
                                             i + 1, GETBIT(gs->btdata, i), 0, 0);
-                        TRACE2("button bit: %d %d\n", i, GETBIT(gs->btdata, i));
-                }
-        }
-        if (GETBIT(gs->type, GS_MOVE)) {
-                xf86PostMotionEvent(local->dev, 0, 0, 2,
-                                    gs->dx, gs->dy);
-                TRACE2("motion: %d %d\n", gs->dx, gs->dy);
         }
+        if (GETBIT(gs->type, GS_MOVE))
+                xf86PostMotionEvent(local->dev, 0, 0, 2, gs->dx, gs->dy);
+
         if (GETBIT(gs->type, GS_VSCROLL)) {
-                button_scroll(local, 4, 5, &vscroll, vscrollstep, gs->dy);
-                TRACE1("vscroll: %d\n", gs->dy);
+                int step = 1 + vscroll_fraction * get_cap_ysize(caps);
+                button_scroll(local, 4, 5, &vscroll, step, gs->dy);
         }
         if (GETBIT(gs->type, GS_HSCROLL)) {
-                button_scroll(local, 6, 7, &hscroll, hscrollstep, gs->dx);
-                TRACE1("hscroll: %d\n", gs->dx);
+                int step = 1 + hscroll_fraction * get_cap_xsize(caps);
+                button_scroll(local, 6, 7, &hscroll, step, gs->dx);
         }
         if (GETBIT(gs->type, GS_VSWIPE)) {
-                button_scroll(local, 8, 9, &vswipe, vswipestep, gs->dy);
-                TRACE1("vswipe: %d\n", gs->dy);
+                int step = 1 + vswipe_fraction * get_cap_ysize(caps);
+                button_scroll(local, 8, 9, &vswipe, step, gs->dy);
         }
         if (GETBIT(gs->type, GS_HSWIPE)) {
-                button_scroll(local, 10, 11, &hswipe, hswipestep, gs->dx);
-                TRACE1("hswipe: %d\n", gs->dx);
+                int step = 1 + hswipe_fraction * get_cap_xsize(caps);
+                button_scroll(local, 10, 11, &hswipe, step, gs->dx);
         }
         if (GETBIT(gs->type, GS_SCALE)) {
-                button_scroll(local, 12, 13, &scale, scalestep, gs->scale);
-                TRACE1("scale: %d\n", gs->scale);
+                int step = 1 + scale_fraction * get_cap_xsize(caps);
+                button_scroll(local, 12, 13, &scale, step, gs->scale);
         }
         if (GETBIT(gs->type, GS_ROTATE)) {
-                button_scroll(local, 14, 15, &rot, rotstep, gs->rot);
-                TRACE1("rotate: %d\n", gs->rot);
+                int step = 1 + rot_fraction * get_cap_xsize(caps);
+                button_scroll(local, 14, 15, &rot, step, gs->rot);
         }
 }