[PATCH libinput 3/3] test: use poll instead of a busy wait

Peter Hutterer peter.hutterer at who-t.net
Thu Jul 2 00:46:48 PDT 2015


Drop the busy loop we had waiting for an event to appear and just call poll on
the libinput fd.

This actually makes the tests more correct, if we now time out where we didn't
before it means we're not setting the timers correctly.

Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
 test/litest.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/test/litest.c b/test/litest.c
index a91207e..df521fe 100644
--- a/test/litest.c
+++ b/test/litest.c
@@ -1560,6 +1560,7 @@ litest_wait_for_event_of_type(struct libinput *li, ...)
 	enum libinput_event_type types[32] = {LIBINPUT_EVENT_NONE};
 	size_t ntypes = 0;
 	enum libinput_event_type type;
+	struct pollfd fds;
 
 	va_start(args, li);
 	type = va_arg(args, int);
@@ -1571,12 +1572,16 @@ litest_wait_for_event_of_type(struct libinput *li, ...)
 	}
 	va_end(args);
 
+	fds.fd = libinput_get_fd(li);
+	fds.events = POLLIN;
+	fds.revents = 0;
+
 	while (1) {
 		size_t i;
 		struct libinput_event *event;
 
 		while ((type = libinput_next_event_type(li)) == LIBINPUT_EVENT_NONE) {
-			msleep(10);
+			poll(&fds, 1, -1);
 			libinput_dispatch(li);
 		}
 
-- 
2.4.3



More information about the wayland-devel mailing list