test: don't set the log handler 10 times
authorPeter Hutterer <peter.hutterer@who-t.net>
Thu, 11 Feb 2021 05:31:12 +0000 (15:31 +1000)
committerPeter Hutterer <peter.hutterer@who-t.net>
Thu, 11 Feb 2021 05:53:53 +0000 (15:53 +1000)
Once is enough, we don't need to set it inside the loop.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
test/test-misc.c

index a48398ac0ea513570f7fe96d5620e8c15aca94b7..7d4a94a659448d02af2292628caf93a55e5f0f1d 100644 (file)
@@ -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);