Switch a bunch of internal functions from int to bool

All these effectively returned bools anyway, switch the signature over to be
less ambiguous.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Yong Bakos <ybakos@humanoriented.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>

9 files changed, 74 insertions, 74 deletions

diff --git a/src/evdev-mt-touchpad-buttons.c b/src/evdev-mt-touchpad-buttons.c
index 3c47746..c7142c9 100644
--- a/src/evdev-mt-touchpad-buttons.c
+++ b/src/evdev-mt-touchpad-buttons.c
@@ -1186,7 +1186,7 @@ tp_post_button_events(struct tp_dispatch *tp, uint64_t time)
 		return tp_post_physical_buttons(tp, time);
 }
 
-int
+bool
 tp_button_touch_active(const struct tp_dispatch *tp,
 		       const struct tp_touch *t)
 {
diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c
index fe0d4a4..7ee86a9 100644
--- a/src/evdev-mt-touchpad.c
+++ b/src/evdev-mt-touchpad.c
@@ -541,7 +541,7 @@ tp_pin_fingers(struct tp_dispatch *tp)
 	}
 }
 
-int
+bool
 tp_touch_active(const struct tp_dispatch *tp, const struct tp_touch *t)
 {
 	return (t->state == TOUCH_BEGIN || t->state == TOUCH_UPDATE) &&
@@ -1673,7 +1673,7 @@ tp_sync_touch(struct tp_dispatch *tp,
 	libevdev_fetch_slot_value(evdev, slot, ABS_MT_DISTANCE, &t->distance);
 }
 
-static int
+static bool
 tp_init_slots(struct tp_dispatch *tp,
 	      struct evdev_device *device)
 {
@@ -1736,7 +1736,7 @@ tp_init_slots(struct tp_dispatch *tp,
 	tp->ntouches = max(tp->num_slots, n_btn_tool_touches);
 	tp->touches = calloc(tp->ntouches, sizeof(struct tp_touch));
 	if (!tp->touches)
-		return -1;
+		return false;
 
 	for (i = 0; i < tp->ntouches; i++)
 		tp_init_touch(tp, &tp->touches[i]);
@@ -1747,7 +1747,7 @@ tp_init_slots(struct tp_dispatch *tp,
 	for (i = 1; i < tp->num_slots; i++)
 		tp_sync_touch(tp, device, &tp->touches[i], i);
 
-	return 0;
+	return true;
 }
 
 static uint32_t
@@ -1775,7 +1775,7 @@ tp_accel_config_get_default_profile(struct libinput_device *libinput_device)
 	return LIBINPUT_CONFIG_ACCEL_PROFILE_NONE;
 }
 
-static int
+static bool
 tp_init_accel(struct tp_dispatch *tp)
 {
 	struct evdev_device *device = tp->device;
@@ -1802,7 +1802,7 @@ tp_init_accel(struct tp_dispatch *tp)
 		filter = create_pointer_accelerator_filter_touchpad(tp->device->dpi);
 
 	if (!filter)
-		return -1;
+		return false;
 
 	evdev_device_init_pointer_acceleration(tp->device, filter);
 
@@ -1813,7 +1813,7 @@ tp_init_accel(struct tp_dispatch *tp)
 	device->pointer.config.get_profile = tp_accel_config_get_profile;
 	device->pointer.config.get_default_profile = tp_accel_config_get_default_profile;
 
-	return 0;
+	return true;
 }
 
 static uint32_t
@@ -2186,8 +2186,8 @@ tp_init(struct tp_dispatch *tp,
 
 	tp_init_default_resolution(tp, device);
 
-	if (tp_init_slots(tp, device) != 0)
-		return -1;
+	if (!tp_init_slots(tp, device))
+		return false;
 
 	width = device->abs.dimensions.x;
 	height = device->abs.dimensions.y;
@@ -2200,8 +2200,8 @@ tp_init(struct tp_dispatch *tp,
 
 	tp_init_hysteresis(tp);
 
-	if (tp_init_accel(tp) != 0)
-		return -1;
+	if (!tp_init_accel(tp))
+		return false;
 
 	tp_init_tap(tp);
 	tp_init_buttons(tp, device);
@@ -2216,7 +2216,7 @@ tp_init(struct tp_dispatch *tp,
 	if (tp->gesture.enabled)
 		device->seat_caps |= EVDEV_DEVICE_GESTURE;
 
-	return 0;
+	return true;
 }
 
 static uint32_t
@@ -2323,7 +2323,7 @@ evdev_mt_touchpad_create(struct evdev_device *device)
 	if (!tp)
 		return NULL;
 
-	if (tp_init(tp, device) != 0) {
+	if (!tp_init(tp, device)) {
 		tp_interface_destroy(&tp->base);
 		return NULL;
 	}
diff --git a/src/evdev-mt-touchpad.h b/src/evdev-mt-touchpad.h
index 24912aa..86dd048 100644
--- a/src/evdev-mt-touchpad.h
+++ b/src/evdev-mt-touchpad.h
@@ -422,7 +422,7 @@ tp_filter_motion_unaccelerated(struct tp_dispatch *tp,
 			       const struct normalized_coords *unaccelerated,
 			       uint64_t time);
 
-int
+bool
 tp_touch_active(const struct tp_dispatch *tp, const struct tp_touch *t);
 
 int
@@ -460,7 +460,7 @@ tp_post_button_events(struct tp_dispatch *tp, uint64_t time);
 void
 tp_button_handle_state(struct tp_dispatch *tp, uint64_t time);
 
-int
+bool
 tp_button_touch_active(const struct tp_dispatch *tp,
 		       const struct tp_touch *t);
 
diff --git a/src/evdev-tablet.c b/src/evdev-tablet.c
index a47f9e8..f1dce27 100644
--- a/src/evdev-tablet.c
+++ b/src/evdev-tablet.c
@@ -76,7 +76,7 @@ tablet_force_button_presses(struct tablet_dispatch *tablet)
 	}
 }
 
-static int
+static bool
 tablet_device_has_axis(struct tablet_dispatch *tablet,
 		       enum libinput_tablet_tool_axis axis)
 {
diff --git a/src/evdev.c b/src/evdev.c
index 1af8d87..aeb21eb 100644
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -97,7 +97,7 @@ hw_set_key_down(struct evdev_device *device, int code, int pressed)
 	long_set_bit_state(device->hw_key_mask, code, pressed);
 }
 
-static int
+static bool
 hw_is_key_down(struct evdev_device *device, int code)
 {
 	return long_bit_is_set(device->hw_key_mask, code);
@@ -1536,7 +1536,7 @@ evdev_device_dispatch(void *data)
 	}
 }
 
-static inline int
+static inline bool
 evdev_init_accel(struct evdev_device *device,
 		 enum libinput_config_accel_profile which)
 {
@@ -1552,11 +1552,11 @@ evdev_init_accel(struct evdev_device *device,
 		filter = create_pointer_accelerator_filter_linear(device->dpi);
 
 	if (!filter)
-		return -1;
+		return false;
 
 	evdev_device_init_pointer_acceleration(device, filter);
 
-	return 0;
+	return true;
 }
 
 static int
@@ -1619,7 +1619,7 @@ evdev_accel_config_set_profile(struct libinput_device *libinput_device,
 	speed = filter_get_speed(filter);
 	device->pointer.filter = NULL;
 
-	if (evdev_init_accel(device, profile) == 0) {
+	if (evdev_init_accel(device, profile)) {
 		evdev_accel_config_set_speed(libinput_device, speed);
 		filter_destroy(filter);
 	} else {
@@ -1671,7 +1671,7 @@ evdev_device_init_pointer_acceleration(struct evdev_device *device,
 	}
 }
 
-static inline int
+static inline bool
 evdev_need_mtdev(struct evdev_device *device)
 {
 	struct libevdev *evdev = device->evdev;
@@ -1820,7 +1820,7 @@ evdev_read_model_flags(struct evdev_device *device)
 	return model_flags;
 }
 
-static inline int
+static inline bool
 evdev_read_attr_res_prop(struct evdev_device *device,
 			 size_t *xres,
 			 size_t *yres)
@@ -1837,7 +1837,7 @@ evdev_read_attr_res_prop(struct evdev_device *device,
 	return parse_dimension_property(res_prop, xres, yres);
 }
 
-static inline int
+static inline bool
 evdev_read_attr_size_prop(struct evdev_device *device,
 			  size_t *size_x,
 			  size_t *size_y)
@@ -1957,14 +1957,14 @@ evdev_fix_android_mt(struct evdev_device *device)
 		      libevdev_get_abs_info(evdev, ABS_MT_POSITION_Y));
 }
 
-static inline int
+static inline bool
 evdev_check_min_max(struct evdev_device *device, unsigned int code)
 {
 	struct libevdev *evdev = device->evdev;
 	const struct input_absinfo *absinfo;
 
 	if (!libevdev_has_event_code(evdev, EV_ABS, code))
-		return 0;
+		return true;
 
 	absinfo = libevdev_get_abs_info(evdev, code);
 	if (absinfo->minimum == absinfo->maximum) {
@@ -1987,14 +1987,14 @@ evdev_check_min_max(struct evdev_device *device, unsigned int code)
 				       "Device '%s' has min == max on %s\n",
 				       device->devname,
 				       libevdev_event_code_get_name(EV_ABS, code));
-			return -1;
+			return false;
 		}
 	}
 
-	return 0;
+	return true;
 }
 
-static int
+static bool
 evdev_reject_device(struct evdev_device *device)
 {
 	struct libinput *libinput = evdev_libinput_context(device);
@@ -2004,16 +2004,16 @@ evdev_reject_device(struct evdev_device *device)
 
 	if (libevdev_has_event_code(evdev, EV_ABS, ABS_X) ^
 	    libevdev_has_event_code(evdev, EV_ABS, ABS_Y))
-		return -1;
+		return true;
 
 	if (libevdev_has_event_code(evdev, EV_REL, REL_X) ^
 	    libevdev_has_event_code(evdev, EV_REL, REL_Y))
-		return -1;
+		return true;
 
 	if (!evdev_is_fake_mt_device(device) &&
 	    libevdev_has_event_code(evdev, EV_ABS, ABS_MT_POSITION_X) ^
 	    libevdev_has_event_code(evdev, EV_ABS, ABS_MT_POSITION_Y))
-		return -1;
+		return true;
 
 	if (libevdev_has_event_code(evdev, EV_ABS, ABS_X)) {
 		absx = libevdev_get_abs_info(evdev, ABS_X);
@@ -2022,7 +2022,7 @@ evdev_reject_device(struct evdev_device *device)
 		    (absx->resolution != 0 && absy->resolution == 0)) {
 			log_bug_kernel(libinput,
 				       "Kernel has only x or y resolution, not both.\n");
-			return -1;
+			return true;
 		}
 	}
 
@@ -2034,7 +2034,7 @@ evdev_reject_device(struct evdev_device *device)
 		    (absx->resolution != 0 && absy->resolution == 0)) {
 			log_bug_kernel(libinput,
 				       "Kernel has only x or y MT resolution, not both.\n");
-			return -1;
+			return true;
 		}
 	}
 
@@ -2045,15 +2045,15 @@ evdev_reject_device(struct evdev_device *device)
 		case ABS_MT_TOOL_TYPE:
 			break;
 		default:
-			if (evdev_check_min_max(device, code) == -1)
-				return -1;
+			if (!evdev_check_min_max(device, code))
+				return true;
 		}
 	}
 
-	return 0;
+	return false;
 }
 
-static int
+static bool
 evdev_configure_mt_device(struct evdev_device *device)
 {
 	struct libevdev *evdev = device->evdev;
@@ -2064,7 +2064,7 @@ evdev_configure_mt_device(struct evdev_device *device)
 
 	if (!libevdev_has_event_code(evdev, EV_ABS, ABS_MT_POSITION_X) ||
 	    !libevdev_has_event_code(evdev, EV_ABS, ABS_MT_POSITION_Y))
-		 return 0;
+		 return true;
 
 	if (evdev_fix_abs_resolution(device,
 				     ABS_MT_POSITION_X,
@@ -2085,7 +2085,7 @@ evdev_configure_mt_device(struct evdev_device *device)
 	if (evdev_need_mtdev(device)) {
 		device->mtdev = mtdev_new_open(device->fd);
 		if (!device->mtdev)
-			return -1;
+			return false;
 
 		/* pick 10 slots as default for type A
 		   devices. */
@@ -2098,7 +2098,7 @@ evdev_configure_mt_device(struct evdev_device *device)
 
 	slots = calloc(num_slots, sizeof(struct mt_slot));
 	if (!slots)
-		return -1;
+		return false;
 
 	for (slot = 0; slot < num_slots; ++slot) {
 		slots[slot].seat_slot = -1;
@@ -2123,10 +2123,10 @@ evdev_configure_mt_device(struct evdev_device *device)
 		device->mt.hysteresis_margin.y = device->abs.absinfo_y->fuzz/2;
 	}
 
-	return 0;
+	return true;
 }
 
-static int
+static bool
 evdev_configure_device(struct evdev_device *device)
 {
 	struct libinput *libinput = evdev_libinput_context(device);
@@ -2142,7 +2142,7 @@ evdev_configure_device(struct evdev_device *device)
 		log_info(libinput,
 			 "input device '%s', %s not tagged as input device\n",
 			 device->devname, devnode);
-		return -1;
+		return false;
 	}
 
 	log_info(libinput,
@@ -2162,7 +2162,7 @@ evdev_configure_device(struct evdev_device *device)
 		log_info(libinput,
 			 "input device '%s', %s is an accelerometer, ignoring\n",
 			 device->devname, devnode);
-		return -1;
+		return false;
 	}
 
 	/* libwacom *adds* TABLET, TOUCHPAD but leaves JOYSTICK in place, so
@@ -2171,14 +2171,14 @@ evdev_configure_device(struct evdev_device *device)
 		log_info(libinput,
 			 "input device '%s', %s is a joystick, ignoring\n",
 			 device->devname, devnode);
-		return -1;
+		return false;
 	}
 
-	if (evdev_reject_device(device) == -1) {
+	if (evdev_reject_device(device)) {
 		log_info(libinput,
 			 "input device '%s', %s was rejected.\n",
 			 device->devname, devnode);
-		return -1;
+		return false;
 	}
 
 	if (!evdev_is_fake_mt_device(device))
@@ -2198,8 +2198,8 @@ evdev_configure_device(struct evdev_device *device)
 
 		if (evdev_is_fake_mt_device(device)) {
 			udev_tags &= ~EVDEV_UDEV_TAG_TOUCHSCREEN;
-		} else if (evdev_configure_mt_device(device) == -1) {
-			return -1;
+		} else if (!evdev_configure_mt_device(device)) {
+			return false;
 		}
 	}
 
@@ -2217,7 +2217,7 @@ evdev_configure_device(struct evdev_device *device)
 		log_info(libinput,
 			 "input device '%s', %s is a tablet pad\n",
 			 device->devname, devnode);
-		return device->dispatch == NULL ? -1 : 0;
+		return device->dispatch != NULL;
 
 	} else if ((udev_tags & tablet_tags) == EVDEV_UDEV_TAG_TABLET) {
 		device->dispatch = evdev_tablet_create(device);
@@ -2225,7 +2225,7 @@ evdev_configure_device(struct evdev_device *device)
 		log_info(libinput,
 			 "input device '%s', %s is a tablet\n",
 			 device->devname, devnode);
-		return device->dispatch == NULL ? -1 : 0;
+		return device->dispatch != NULL;
 	}
 
 	if (udev_tags & EVDEV_UDEV_TAG_TOUCHPAD) {
@@ -2234,7 +2234,7 @@ evdev_configure_device(struct evdev_device *device)
 			 "input device '%s', %s is a touchpad\n",
 			 device->devname, devnode);
 
-		return device->dispatch == NULL ? -1 : 0;
+		return device->dispatch != NULL;
 	}
 
 	if (udev_tags & EVDEV_UDEV_TAG_MOUSE ||
@@ -2283,14 +2283,14 @@ evdev_configure_device(struct evdev_device *device)
 	if (device->seat_caps & EVDEV_DEVICE_POINTER &&
 	    libevdev_has_event_code(evdev, EV_REL, REL_X) &&
 	    libevdev_has_event_code(evdev, EV_REL, REL_Y) &&
-	    evdev_init_accel(device, LIBINPUT_CONFIG_ACCEL_PROFILE_ADAPTIVE) == -1) {
+	    !evdev_init_accel(device, LIBINPUT_CONFIG_ACCEL_PROFILE_ADAPTIVE)) {
 		log_error(libinput,
 			  "failed to initialize pointer acceleration for %s\n",
 			  device->devname);
-		return -1;
+		return false;
 	}
 
-	return 0;
+	return true;
 }
 
 static void
@@ -2346,7 +2346,7 @@ out:
 	return rc;
 }
 
-static int
+static bool
 evdev_set_device_group(struct evdev_device *device,
 		       struct udev_device *udev_device)
 {
@@ -2362,14 +2362,14 @@ evdev_set_device_group(struct evdev_device *device,
 	if (!group) {
 		group = libinput_device_group_create(libinput, udev_group);
 		if (!group)
-			return 1;
+			return false;
 		libinput_device_set_device_group(&device->base, group);
 		libinput_device_group_unref(group);
 	} else {
 		libinput_device_set_device_group(&device->base, group);
 	}
 
-	return 0;
+	return true;
 }
 
 static inline void
@@ -2493,7 +2493,7 @@ evdev_device_create(struct libinput_seat *seat,
 
 	evdev_pre_configure_model_quirks(device);
 
-	if (evdev_configure_device(device) == -1)
+	if (!evdev_configure_device(device))
 		goto err;
 
 	if (device->seat_caps == 0) {
@@ -2512,7 +2512,7 @@ evdev_device_create(struct libinput_seat *seat,
 	if (!device->source)
 		goto err;
 
-	if (evdev_set_device_group(device, udev_device))
+	if (!evdev_set_device_group(device, udev_device))
 		goto err;
 
 	list_insert(seat->devices_list.prev, &device->base.link);
@@ -2638,7 +2638,7 @@ evdev_device_calibrate(struct evdev_device *device,
 	matrix_mult(&device->abs.calibration, &transform, &scale);
 }
 
-int
+bool
 evdev_device_has_capability(struct evdev_device *device,
 			    enum libinput_device_capability capability)
 {
@@ -2656,7 +2656,7 @@ evdev_device_has_capability(struct evdev_device *device,
 	case LIBINPUT_DEVICE_CAP_TABLET_PAD:
 		return !!(device->seat_caps & EVDEV_DEVICE_TABLET_PAD);
 	default:
-		return 0;
+		return false;
 	}
 }
 
diff --git a/src/evdev.h b/src/evdev.h
index 9ddd8b3..d70dd34 100644
--- a/src/evdev.h
+++ b/src/evdev.h
@@ -366,7 +366,7 @@ void
 evdev_device_calibrate(struct evdev_device *device,
 		       const float calibration[6]);
 
-int
+bool
 evdev_device_has_capability(struct evdev_device *device,
 			    enum libinput_device_capability capability);
 
diff --git a/src/libinput-private.h b/src/libinput-private.h
index b325707..472f6f3 100644
--- a/src/libinput-private.h
+++ b/src/libinput-private.h
@@ -665,7 +665,7 @@ normalized_length(struct normalized_coords norm)
 	return hypot(norm.x, norm.y);
 }
 
-static inline int
+static inline bool
 normalized_is_zero(struct normalized_coords norm)
 {
 	return norm.x == 0.0 && norm.y == 0.0;
diff --git a/src/libinput-util.c b/src/libinput-util.c
index d35d702..4b90fbb 100644
--- a/src/libinput-util.c
+++ b/src/libinput-util.c
@@ -65,7 +65,7 @@ list_remove(struct list *elm)
 	elm->prev = NULL;
 }
 
-int
+bool
 list_empty(const struct list *list)
 {
 	return list->next == list;
diff --git a/src/libinput-util.h b/src/libinput-util.h
index 0848040..0c9a6cf 100644
--- a/src/libinput-util.h
+++ b/src/libinput-util.h
@@ -62,7 +62,7 @@ struct list {
 void list_init(struct list *list);
 void list_insert(struct list *list, struct list *elm);
 void list_remove(struct list *elm);
-int list_empty(const struct list *list);
+bool list_empty(const struct list *list);
 
 #ifdef __GNUC__
 #define container_of(ptr, sample, member)				\
@@ -124,7 +124,7 @@ zalloc(size_t size)
  * except that it has been modified to work with arrays of unsigned chars
  */
 
-static inline int
+static inline bool
 bit_is_set(const unsigned char *array, int bit)
 {
 	return !!(array[bit / 8] & (1 << (bit % 8)));
@@ -148,7 +148,7 @@ msleep(unsigned int ms)
 	usleep(ms * 1000);
 }
 
-static inline int
+static inline bool
 long_bit_is_set(const unsigned long *array, int bit)
 {
 	return !!(array[bit / LONG_BITS] & (1LL << (bit % LONG_BITS)));
@@ -175,7 +175,7 @@ long_set_bit_state(unsigned long *array, int bit, int state)
 		long_clear_bit(array, bit);
 }
 
-static inline int
+static inline bool
 long_any_bit_set(unsigned long *array, size_t size)
 {
 	unsigned long i;
@@ -184,8 +184,8 @@ long_any_bit_set(unsigned long *array, size_t size)
 
 	for (i = 0; i < size; i++)
 		if (array[i] != 0)
-			return 1;
-	return 0;
+			return true;
+	return false;
 }
 
 static inline double
@@ -250,7 +250,7 @@ matrix_init_rotate(struct matrix *m, int degrees)
 	m->val[1][1] = c;
 }
 
-static inline int
+static inline bool
 matrix_is_identity(const struct matrix *m)
 {
 	return (m->val[0][0] == 1 &&