touchpad: mark a intentional switch case fallthrough as such
authorPeter Hutterer <peter.hutterer@who-t.net>
Fri, 22 Aug 2014 04:58:48 +0000 (14:58 +1000)
committerPeter Hutterer <peter.hutterer@who-t.net>
Tue, 26 Aug 2014 01:04:42 +0000 (11:04 +1000)
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 <peter.hutterer@who-t.net>
src/evdev-mt-touchpad-tap.c

index 64801a8..fae8f5e 100644 (file)
@@ -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;