Francois Gouget : mstask/tests: Ignore the compare_trigger_state() return value.

Alexandre Julliard julliard at winehq.org
Mon May 16 15:37:57 CDT 2022


Module: wine
Branch: master
Commit: 1017c188f70de56426adb2d5df2db9fcefd264f1
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=1017c188f70de56426adb2d5df2db9fcefd264f1

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Fri May 13 19:59:18 2022 +0200

mstask/tests: Ignore the compare_trigger_state() return value.

What counts is the ok() calls that compare_trigger_state() makes. Its
return value is meaningless.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/mstask/tests/task_trigger.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/dlls/mstask/tests/task_trigger.c b/dlls/mstask/tests/task_trigger.c
index d0f659ef246..e0c1b549ab7 100644
--- a/dlls/mstask/tests/task_trigger.c
+++ b/dlls/mstask/tests/task_trigger.c
@@ -35,7 +35,7 @@ static DWORD obj_refcount(void *obj_to_check)
     return IUnknown_Release(obj);
 }
 
-static BOOL compare_trigger_state(TASK_TRIGGER found_state,
+static void compare_trigger_state(TASK_TRIGGER found_state,
         TASK_TRIGGER expected_state)
 {
     ok(found_state.cbTriggerSize == expected_state.cbTriggerSize,
@@ -105,8 +105,6 @@ static BOOL compare_trigger_state(TASK_TRIGGER found_state,
     ok(found_state.wRandomMinutesInterval == expected_state.wRandomMinutesInterval,
             "wRandomMinutesInterval: Found %d but expected %d\n",
             found_state.wRandomMinutesInterval, expected_state.wRandomMinutesInterval);
-
-    return TRUE;
 }
 
 static void test_SetTrigger_GetTrigger(void)
@@ -342,14 +340,12 @@ static void test_SetTrigger_GetTrigger(void)
     trigger_state.cbTriggerSize = sizeof(trigger_state) - 1;
     hres = ITaskTrigger_GetTrigger(test_trigger, &trigger_state);
     ok(hres == S_OK, "Failed to GetTrigger\n");
-    ok(compare_trigger_state(trigger_state, normal_trigger_state),
-            "Invalid state\n");
+    compare_trigger_state(trigger_state, normal_trigger_state);
     memset(&trigger_state, 0xcf, sizeof(trigger_state));
     trigger_state.cbTriggerSize = 0;
     hres = ITaskTrigger_GetTrigger(test_trigger, &trigger_state);
     ok(hres == S_OK, "Failed to GetTrigger\n");
-    ok(compare_trigger_state(trigger_state, normal_trigger_state),
-            "Invalid state\n");
+    compare_trigger_state(trigger_state, normal_trigger_state);
 
     ITaskTrigger_Release(test_trigger);
     ITask_Release(test_task);




More information about the wine-cvs mailing list