[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