[tools] winetest/dissect: Improve the documentation of the missing start line handling.

Francois Gouget fgouget at codeweavers.com
Thu Jan 28 23:20:10 CST 2021


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 winetest/dissect | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/winetest/dissect b/winetest/dissect
index e9729dbc8..27bcf1704 100755
--- a/winetest/dissect
+++ b/winetest/dissect
@@ -717,23 +717,28 @@ while ($line = <IN>) {
 
         if ($l_dll ne $dll or $l_unit ne $unit)
         {
-            # First close the current test unit taking into account
-            # it may have been polluted by the new one.
+            # Warn about the missing start line in the current test unit box.
             add_test_line("end", "The $l_dll:$l_unit start line is missing (or it is garbled)");
             $extra_failures++;
+
+            # And close the current test unit taking into account
+            # it may have been polluted by the new one.
             $broken = 1;
             close_test_unit(0);
 
-            # Then switch to the new one, warning it's missing a start line,
-            # and that its results may be inconsistent.
+            # Then switch to the new test unit, not for the past lines, but for
+            # those before the next 'start' line. This 'new' test unit may have
+            # inconsistent results too.
             ($dll, $unit, $source, $rev) = ($l_dll, $l_unit, "-", "-");
             %units = ($unit => 1);
             $units_re = $unit;
+            $broken = 1;
 
             $testbox = create_test_unit_box();
+            # Finally, warn about the missing start line in the new test unit
+            # box.
             add_test_line("end", "The $l_dll:$l_unit start line is missing (or it is garbled)");
             $extra_failures++;
-            $broken = 1;
         }
 
         my $class = $l_rc ? "failed" : "";
-- 
2.20.1




More information about the wine-devel mailing list