[PATCH 3/8] evdev: Avoid unintended case drop through

Rob Bradford robert.bradford at intel.com
Tue Oct 9 10:44:31 PDT 2012


From: Rob Bradford <rob at linux.intel.com>

e->code is in the same range for ABS_ and for REL_. As the code currently
stands and for the current values in Linux's input.h there is no risk of a
problem. However just in case it would be wise to break after evaluating the
relative events.

Signed-off-by: Rob Bradford <rob at linux.intel.com>
---
 src/evdev.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/evdev.c b/src/evdev.c
index 8848736..cc5af91 100644
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -195,6 +195,7 @@ is_motion_event(struct input_event *e)
 		case REL_Y:
 			return 1;
 		}
+		break;
 	case EV_ABS:
 		switch (e->code) {
 		case ABS_X:
-- 
1.7.11.2



More information about the wayland-devel mailing list