Fix two signed vs unsigned int warnings
authorPeter Hutterer <peter.hutterer@who-t.net>
Thu, 29 Aug 2013 01:53:59 +0000 (11:53 +1000)
committerPeter Hutterer <peter.hutterer@who-t.net>
Thu, 29 Aug 2013 03:20:07 +0000 (13:20 +1000)
dev->num_slots is -1 if we don't have ABS_MT_SLOT.

Set dev->grabbed to the right field type.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
libevdev/libevdev-int.h
libevdev/libevdev.c

index 2be555d..61c2454 100644 (file)
@@ -99,7 +99,7 @@ struct libevdev {
        int rep_values[REP_CNT];
 
        enum SyncState sync_state;
-       int grabbed;
+       enum libevdev_grab_mode grabbed;
 
        struct input_event *queue;
        size_t queue_size; /**< size of queue in elements */
index 598455e..95d041c 100644 (file)
@@ -936,7 +936,7 @@ libevdev_set_slot_value(struct libevdev *dev, unsigned int slot, unsigned int co
        if (!libevdev_has_event_type(dev, EV_ABS) || !libevdev_has_event_code(dev, EV_ABS, code))
                return -1;
 
-       if (slot >= dev->num_slots || slot >= MAX_SLOTS)
+       if (dev->num_slots == -1 || slot >= (unsigned int)dev->num_slots || slot >= MAX_SLOTS)
                return -1;
 
        if (code > ABS_MT_MAX || code < ABS_MT_MIN)