Now that we've separate handling of the EDGE_NEW vs EDGE states in
tp_edge_scroll_post_events() we can drop the threshold variable, in EDGE_NEW
we always want to check against DEFAULT_SCROLL_THRESHOLD and in the EDGE
state we only want to make sure that the delta != 0.0 which is already
checked later on in tp_edge_scroll_post_events().

Signed-off-by: Hans de Goede <hdego...@redhat.com>
---
 src/evdev-mt-touchpad-edge-scroll.c | 7 +------
 src/evdev-mt-touchpad.h             | 1 -
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/evdev-mt-touchpad-edge-scroll.c 
b/src/evdev-mt-touchpad-edge-scroll.c
index 3adde8c..a7936b6 100644
--- a/src/evdev-mt-touchpad-edge-scroll.c
+++ b/src/evdev-mt-touchpad-edge-scroll.c
@@ -73,7 +73,6 @@ tp_edge_scroll_set_state(struct tp_dispatch *tp,
        switch (state) {
        case EDGE_SCROLL_TOUCH_STATE_NONE:
                t->scroll.edge = EDGE_NONE;
-               t->scroll.threshold = DEFAULT_SCROLL_THRESHOLD;
                break;
        case EDGE_SCROLL_TOUCH_STATE_EDGE_NEW:
                t->scroll.edge = tp_touch_get_edge(tp, t);
@@ -83,7 +82,6 @@ tp_edge_scroll_set_state(struct tp_dispatch *tp,
                                   t->millis + DEFAULT_SCROLL_LOCK_TIMEOUT);
                break;
        case EDGE_SCROLL_TOUCH_STATE_EDGE:
-               t->scroll.threshold = 0.01; /* Do not allow 0.0 events */
                break;
        case EDGE_SCROLL_TOUCH_STATE_AREA:
                t->scroll.edge = EDGE_NONE;
@@ -265,7 +263,6 @@ tp_edge_scroll_init(struct tp_dispatch *tp, struct 
evdev_device *device)
 
        tp_for_each_touch(tp, t) {
                t->scroll.direction = -1;
-               t->scroll.threshold = DEFAULT_SCROLL_THRESHOLD;
                libinput_timer_init(&t->scroll.timer,
                                    device->base.seat->libinput,
                                    tp_edge_scroll_handle_timeout, t);
@@ -367,7 +364,7 @@ tp_edge_scroll_post_events(struct tp_dispatch *tp, uint64_t 
time)
                        tp_normalize_delta(tp,
                                           &initial_dx,
                                           &initial_dy);
-                       if (fabs(*initial_delta) < t->scroll.threshold) {
+                       if (fabs(*initial_delta) < DEFAULT_SCROLL_THRESHOLD) {
                                dx = 0.0;
                                dy = 0.0;
                        } else {
@@ -376,8 +373,6 @@ tp_edge_scroll_post_events(struct tp_dispatch *tp, uint64_t 
time)
                        }
                        break;
                case EDGE_SCROLL_TOUCH_STATE_EDGE:
-                       if (fabs(*delta) < t->scroll.threshold)
-                               *delta = 0.0;
                        break;
                }
 
diff --git a/src/evdev-mt-touchpad.h b/src/evdev-mt-touchpad.h
index 57b3370..f88fc4a 100644
--- a/src/evdev-mt-touchpad.h
+++ b/src/evdev-mt-touchpad.h
@@ -173,7 +173,6 @@ struct tp_touch {
                enum tp_edge_scroll_touch_state edge_state;
                uint32_t edge;
                int direction;
-               double threshold;
                struct libinput_timer timer;
                int32_t initial_x;              /* in device coordinates */
                int32_t initial_y;              /* in device coordinates */
-- 
2.3.1

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to