commit c9785eacca7fe48119547bc7ed476754d8bcf610
author Daniel Stone <daniel@fooishbar.org> Wed May 30 16:31:52 2012 +0100
committer Kristian Høgsberg <krh@bitplanet.net> Thu May 31 15:43:03 2012 -0400
tree daab9cadd0b852786aaffe124862a1839cfc4e44
parent 4dbadb1556f296cf148ed91a1044004ec6c80707

Use enum wl_keyboard_key_state instead of integer

Instead of using a uint32_t for state everywhere (except on the wire,
where that's still the call signature), use the new
wl_keyboard_key_state enum, and explicit comparisons.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

src/compositor.c

diff --git a/src/compositor.c b/src/compositor.c
index 14d1494..7d4069e 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -1790,7 +1790,8 @@
 }
 
 WL_EXPORT void
-notify_key(struct wl_seat *seat, uint32_t time, uint32_t key, uint32_t state)
+notify_key(struct wl_seat *seat, uint32_t time, uint32_t key,
+	   enum wl_keyboard_key_state state)
 {
 	struct weston_seat *ws = (struct weston_seat *) seat;
 	struct weston_compositor *compositor = ws->compositor;
@@ -1801,7 +1802,7 @@
 	uint32_t *k, *end;
 	int mods;
 
-	if (state) {
+	if (state == WL_KEYBOARD_KEY_STATE_PRESSED) {
 		if (compositor->ping_handler && focus)
 			compositor->ping_handler(focus, serial);
 
@@ -1819,7 +1820,7 @@
 			*k = *--end;
 	}
 	seat->keyboard->keys.size = (void *) end - seat->keyboard->keys.data;
-	if (state) {
+	if (state == WL_KEYBOARD_KEY_STATE_PRESSED) {
 		k = wl_array_add(&seat->keyboard->keys, sizeof *k);
 		*k = key;
 	}