Use enum wl_pointer_button_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_pointer_button_state enum, and explicit comparisons.

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

diff --git a/src/evdev-touchpad.c b/src/evdev-touchpad.c
index f658081..d371a65 100644 (file)
--- a/src/evdev-touchpad.c
+++ b/src/evdev-touchpad.c
@@ -444,7 +444,9 @@ process_key(struct touchpad_dispatch *touchpad,
        case BTN_BACK:
        case BTN_TASK:
                notify_button(&device->master->base.seat,
-                             time, e->code, e->value);
+                             time, e->code,
+                             e->value ? WL_POINTER_BUTTON_STATE_PRESSED :
+                                        WL_POINTER_BUTTON_STATE_RELEASED);
                break;
        case BTN_TOOL_PEN:
        case BTN_TOOL_RUBBER:

diff --git a/src/evdev.c b/src/evdev.c
index e73f343..a7162fc 100644 (file)
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -79,7 +79,9 @@ evdev_process_key(struct evdev_input_device *device,
        case BTN_BACK:
        case BTN_TASK:
                notify_button(&device->master->base.seat,
-                             time, e->code, e->value);
+                             time, e->code,
+                             e->value ? WL_POINTER_BUTTON_STATE_PRESSED :
+                                        WL_POINTER_BUTTON_STATE_RELEASED);
                break;
 
        default: