Francois Gouget : winetest: Let the get_subtests() caller report errors.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 75e8f4edb7725d60c20ba8eeb2c637ceede2c4e3
URL:    https://gitlab.winehq.org/wine/wine/-/commit/75e8f4edb7725d60c20ba8eeb2c637c...

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Thu Oct 20 15:19:51 2022 +0200

winetest: Let the get_subtests() caller report errors.

The caller already analyses get_subtests() errors and provides
a detailed error message in the information section of the report
so the report(R_ERROR) pop up / stderr message is redundant.

---

 programs/winetest/main.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/programs/winetest/main.c b/programs/winetest/main.c
index c24af4952c7..74b596b237b 100644
--- a/programs/winetest/main.c
+++ b/programs/winetest/main.c
@@ -815,12 +815,8 @@ get_subtests (const char *tempdir, struct wine_test *test, LPSTR res_name)
 
     if (status)
     {
-        if (status == -2)
-            report (R_ERROR, "Cannot run %s error %u", test->exename, err);
-        else
-            err = status;
         close_temp_file( subname, subfile );
-        return err;
+        return status == -2 ? err : status;
     }
 
     buffer = flush_temp_file( subname, subfile, &total );