Bug 77577

Summary:	Weston claims LIBINPUT_EVENT_TOUCH_UP is an unknown event
Product:	Wayland	Reporter:	U. Artie Eoff <ullysses.a.eoff>
Component:	weston	Assignee:	Wayland bug list <wayland-bugs>
Status:	VERIFIED FIXED	QA Contact:
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description U. Artie Eoff 2014-04-17 14:42:07 UTC

When Weston handles the LIBINPUT_EVENT_TOUCH_UP event, it falls through to the default case and "unknown libinput event 501" gets logged.  This also causes the input event handler to return the wrong result.

libinput (master) heads/master-0-g9c61146
weston (master) heads/master-0-g304996d

Comment 1 U. Artie Eoff 2014-04-17 14:54:42 UTC

Patch: http://lists.freedesktop.org/archives/wayland-devel/2014-April/014282.html

Comment 2 Kristian Høgsberg 2014-04-21 21:36:09 UTC

commit cd9e54537a5cb8acb00e99fbe6eeb74eafdd29fe
Author: U. Artie Eoff <ullysses.a.eoff@intel.com>
Date:   Thu Apr 17 07:53:24 2014 -0700

    libinput-device: break after LIBINPUT_EVENT_TOUCH_UP
    
    We need to break after handling LIBINPUT_EVENT_TOUCH_UP otherwise
    we fall into the default case and end up logging that the event
    is unknown and then return the wrong "handled" result.
    
    Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=77577
    
    Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.