[PATCH] winetest/dissect: Simplify a couple of $units_re initializations.

Francois Gouget fgouget at codeweavers.com
Fri Nov 15 05:50:47 CST 2019


There is no need for a join() when there is no subtest, yet.

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

diff --git a/winetest/dissect b/winetest/dissect
index 13e258e61..6c6c59427 100755
--- a/winetest/dissect
+++ b/winetest/dissect
@@ -603,7 +603,7 @@ while ($line = <IN>) {
 
         ($dll, $unit, $source, $rev) = ($l_dll, $l_unit, $l_source, $l_rev);
         %units = ($unit => 1);
-        $units_re = join("|", keys %units);
+        $units_re = $unit;
 
         $testbox = create_test_unit_box();
         if ($l_type eq "skipped")
@@ -728,7 +728,7 @@ while ($line = <IN>) {
             # and that its results may be inconsistent.
             ($dll, $unit, $source, $rev) = ($l_dll, $l_unit, "-", "-");
             %units = ($unit => 1);
-            $units_re = join("|", keys %units);
+            $units_re = $unit;
 
             $testbox = create_test_unit_box();
             add_test_line("end", "The $l_dll:$l_unit start line is missing (or it is garbled)");
-- 
2.20.1



More information about the wine-devel mailing list