From 27e3897420732d47790932e9c828dd495459bd30 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Fri, 13 Sep 2024 12:56:28 +1000 Subject: [PATCH] test: abort on libinput bugs again This was always intended but a bug prevented the actual abort. strstr returns NULL when we cannot find the substring so we always triggered the first noop condition on bugs. Fixes: bd7b91065b13 ("evdev: warn if our event processing lags by 10ms or more") Part-of: --- test/litest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/litest.c b/test/litest.c index 787e84ae..c2b21bdb 100644 --- a/test/litest.c +++ b/test/litest.c @@ -733,9 +733,9 @@ litest_log_handler(struct libinput *libinput, * short, don't abort if during a valgrind run we get a * negative offset */ if ((RUNNING_ON_VALGRIND && in_debugger) || - !strstr(format, "scheduled expiry is in the past")) { + strstr(format, "scheduled expiry is in the past")) { /* noop */ - } else if (!strstr(format, "event processing lagging behind")) { + } else if (strstr(format, "event processing lagging behind")) { /* noop */ } else { litest_abort_msg("libinput bug triggered, aborting.\n");