[PATCH] libinput-device: Remove unnecessary function call

Chris Michael cpmichael at osg.samsung.com
Mon Feb 22 13:22:11 UTC 2016


When we handle keyboard key events, we already retrieve the key state
at the top of this function, so there is no real need to call the same
libinput function again as we can just reuse the 'key_state' variable
that we have above.

Signed-off-by: Chris Michael <cpmichael at osg.samsung.com>
---
 src/libinput-device.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/libinput-device.c b/src/libinput-device.c
index 78b0ac9..86e61a0 100644
--- a/src/libinput-device.c
+++ b/src/libinput-device.c
@@ -76,8 +76,7 @@ handle_keyboard_key(struct libinput_device *libinput_device,
 	notify_key(device->seat,
 		   libinput_event_keyboard_get_time(keyboard_event),
 		   libinput_event_keyboard_get_key(keyboard_event),
-		   libinput_event_keyboard_get_key_state(keyboard_event),
-		   STATE_UPDATE_AUTOMATIC);
+                   key_state, STATE_UPDATE_AUTOMATIC);
 }
 
 static bool
-- 
2.7.0



More information about the wayland-devel mailing list