Francois Gouget : winetest/gather: Remove support for obsolete statuses.

Alexandre Julliard julliard at winehq.org
Tue Jan 16 11:32:25 CST 2018


Module: tools
Branch: master
Commit: 04404905e79325f5467dec0a7aa69536a459e248
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=04404905e79325f5467dec0a7aa69536a459e248

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Jan 16 04:05:25 2018 +0100

winetest/gather: Remove support for obsolete statuses.

dissect is now responsible for detecting crashes and reporting them as
'failed crash' in the summary file. So gather no longer needs to assume
negative exit codes identify crashes.
'failed -2' should not happen at all.

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

---

 winetest/gather | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/winetest/gather b/winetest/gather
index fbf0397..ac6c350 100755
--- a/winetest/gather
+++ b/winetest/gather
@@ -633,8 +633,6 @@ sub get_result_title($$)
     if ($result->{status} eq "missing") { return "Test did not run for an unknown reason"; }
     if ($result->{status} eq "mixed") { return "Mixed results"; }
     if ($result->{status} eq "crash") { return "Test crashed"; }
-    if ($result->{status} eq "-2") { return "Test failed"; }
-    if ($result->{status} =~ /^-/) { return "Test crashed"; }
     if ($result->{status} eq "258") { return "Test timed out"; }
     return "";
 }
@@ -701,8 +699,6 @@ sub singletest($$$) {
     else
     {
         $msg = $status eq "crash" ? "crashed" :
-               $status eq "-2" ? "failed" :
-               $status =~ /^-/ ? "crashed" :
                $status eq "258" ? "timeout": "failed";
     }
     $title = join(" | ", $prefix, $title);




More information about the wine-cvs mailing list