tp_post_twofinger_scroll: Move setting of t->is_pointer = false up a bit

There is no need to loop over the touch points twice.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c
index 04ea93c6ae5031eae6e1e5c4f5505f62de8236bb..ced92378b3912d40ab9ee1e708f86bb597aaf9a7 100644 (file)
--- a/src/evdev-mt-touchpad.c
+++ b/src/evdev-mt-touchpad.c
@@ -453,6 +453,8 @@ tp_post_twofinger_scroll(struct tp_dispatch *tp, uint64_t time)
                        dx += tmpx;
                        dy += tmpy;
                }
+               /* Stop spurious MOTION events at the end of scrolling */
+               t->is_pointer = false;
        }
 
        if (nchanged == 0)
@@ -470,10 +472,6 @@ tp_post_twofinger_scroll(struct tp_dispatch *tp, uint64_t time)
        if (dx <= -3.0 || dx >= 3.0)
                tp->scroll.direction |= (1 << LIBINPUT_POINTER_AXIS_SCROLL_HORIZONTAL);
 
-       /* Stop spurious MOTION events at the end of scrolling */
-       tp_for_each_touch(tp, t)
-               t->is_pointer = false;
-
        if (dy != 0.0 &&
            (tp->scroll.direction & (1 << LIBINPUT_POINTER_AXIS_SCROLL_VERTICAL))) {
                pointer_notify_axis(&tp->device->base,