From 1d714e68df5cce5c4ff26a154a9c3c00bed49c37 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Thu, 11 Feb 2021 15:31:12 +1000 Subject: [PATCH] test: don't set the log handler 10 times Once is enough, we don't need to set it inside the loop. Signed-off-by: Peter Hutterer --- test/test-misc.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/test/test-misc.c b/test/test-misc.c index a48398ac..7d4a94a6 100644 --- a/test/test-misc.c +++ b/test/test-misc.c @@ -715,18 +715,17 @@ START_TEST(timer_delay_bug_warning) old_user_data = libinput_get_user_data(li); litest_drain_events(li); + libinput_set_user_data(li, &warning_triggered); + libinput_log_set_handler(li, timer_delay_warning); + for (int i = 0; i < 10; i++) { litest_button_click(dev, BTN_LEFT, true); libinput_dispatch(li); litest_button_click(dev, BTN_LEFT, false); msleep(11); - - libinput_set_user_data(li, &warning_triggered); - libinput_log_set_handler(li, timer_delay_warning); libinput_dispatch(li); } - ck_assert_int_ge(warning_triggered, 1); litest_restore_log_handler(li); libinput_set_user_data(li, old_user_data);