dinput/tests: Set the callback event after setting "impl->invoked".

Fixes an intermittent test failure. Signed-off-by: Zebediah Figura <zfigura@codeweavers.com> Signed-off-by: Rémi Bernon <rbernon@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

dinput/tests: Set the callback event after setting "impl->invoked".
887f6891 · Zebediah Figura · Alexandre Julliard · 982a5b9f · 887f6891 · 887f6891
Commit 887f6891 authored May 02, 2022 by Zebediah Figura Committed by Alexandre Julliard May 03, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

force_feedback.c dlls/dinput/tests/force_feedback.c +1 -1

hotplug.c dlls/dinput/tests/hotplug.c +1 -1

joystick8.c dlls/dinput/tests/joystick8.c +1 -1

No files found.
--- a/dlls/dinput/tests/force_feedback.c
+++ b/dlls/dinput/tests/force_feedback.c
@@ -4437,8 +4437,8 @@ static HRESULT WINAPI controller_handler_Invoke( IEventHandler_RawGameController
    trace( "iface %p, sender %p, controller %p\n", iface, sender, controller );
    ok( sender == NULL, "got sender %p\n", sender );
-    SetEvent( impl->event );
    impl->invoked = TRUE;
+    SetEvent( impl->event );
    return S_OK;
 }

--- a/dlls/dinput/tests/hotplug.c
+++ b/dlls/dinput/tests/hotplug.c
@@ -526,8 +526,8 @@ static HRESULT WINAPI controller_handler_Invoke( IEventHandler_RawGameController
    trace( "iface %p, sender %p, controller %p\n", iface, sender, controller );
    ok( sender == NULL, "got sender %p\n", sender );
-    SetEvent( impl->event );
    impl->invoked = TRUE;
+    SetEvent( impl->event );
    return S_OK;
 }

--- a/dlls/dinput/tests/joystick8.c
+++ b/dlls/dinput/tests/joystick8.c
@@ -3845,8 +3845,8 @@ static HRESULT WINAPI controller_handler_Invoke( IEventHandler_RawGameController
    trace( "iface %p, sender %p, controller %p\n", iface, sender, controller );
    ok( sender == NULL, "got sender %p\n", sender );
-    SetEvent( impl->event );
    impl->invoked = TRUE;
+    SetEvent( impl->event );
    return S_OK;
 }