[PATCH libinput 7/7] touchpad: Make tap code follow state machine diagram part 3
Peter Hutterer
peter.hutterer at who-t.net
Mon Nov 3 21:51:31 PST 2014
On Sun, Sep 28, 2014 at 01:22:23PM +0200, Hans de Goede wrote:
> We should only mark touches dead on a button click if we're dealing with a
> clickpad.
>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>
for the series, but it conflicts because of the missing patch. Once I merge
that, I'll merge this series too (with a couple of typos in the commit msgs
fixed). Thanks for all the work!
Cheers,
Peter
> ---
> src/evdev-mt-touchpad-tap.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/evdev-mt-touchpad-tap.c b/src/evdev-mt-touchpad-tap.c
> index e9dc147..ce88ff4 100644
> --- a/src/evdev-mt-touchpad-tap.c
> +++ b/src/evdev-mt-touchpad-tap.c
> @@ -562,7 +562,8 @@ tp_tap_handle_state(struct tp_dispatch *tp, uint64_t time)
> if (!t->dirty || t->state == TOUCH_NONE)
> continue;
>
> - if (tp->queued & TOUCHPAD_EVENT_BUTTON_PRESS)
> + if (tp->buttons.is_clickpad &&
> + tp->queued & TOUCHPAD_EVENT_BUTTON_PRESS)
> t->tap.state = TAP_TOUCH_STATE_DEAD;
>
> if (t->state == TOUCH_BEGIN) {
> --
> 2.1.0
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
More information about the wayland-devel
mailing list