libinput-device: Remove unnecessary function call
authorChris Michael <cpmichael@osg.samsung.com>
Mon, 22 Feb 2016 13:47:24 +0000 (08:47 -0500)
committerPekka Paalanen <pekka.paalanen@collabora.co.uk>
Fri, 26 Feb 2016 13:30:16 +0000 (15:30 +0200)
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@osg.samsung.com>
Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
src/libinput-device.c

index 78b0ac9..f8b614a 100644 (file)
@@ -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