[PATCH] winetest/gather: Remove support for obsolete statuses.

Francois Gouget fgouget at codeweavers.com
Mon Jan 15 21:05:25 CST 2018


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>
---

The obsolete statuses were removed over 6 months ago (see commit 
ee648b52fea5) so we no longer have any file containing them. This means 
it should now be safe to remove support for them.

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

diff --git a/winetest/gather b/winetest/gather
index fbf03975..ac6c3501 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);
-- 
2.15.1




More information about the wine-devel mailing list