From 7b66f16c2a30deb9ab50b3111b9b2611ffab27d6 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Fri, 22 Aug 2014 14:58:48 +1000 Subject: [PATCH] touchpad: mark a intentional switch case fallthrough as such Both motion and timeout expiry transition into the TOUCH_2_HOLD state, but only for motion do we need to cancel the current timeout. Found by Coverity. Signed-off-by: Peter Hutterer --- src/evdev-mt-touchpad-tap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/evdev-mt-touchpad-tap.c b/src/evdev-mt-touchpad-tap.c index 64801a81..fae8f5e8 100644 --- a/src/evdev-mt-touchpad-tap.c +++ b/src/evdev-mt-touchpad-tap.c @@ -253,6 +253,7 @@ tp_tap_touch2_handle_event(struct tp_dispatch *tp, break; case TAP_EVENT_MOTION: tp_tap_clear_timer(tp); + /* fallthrough */ case TAP_EVENT_TIMEOUT: tp->tap.state = TAP_STATE_TOUCH_2_HOLD; break; -- 2.34.1