Francois Gouget : testbot/LogUtils: Rename the $ExtraGroup variable in ParseWineTestReport().

Alexandre Julliard julliard at winehq.org
Tue Feb 16 15:52:55 CST 2021


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Feb 16 15:55:11 2021 +0100

testbot/LogUtils: Rename the $ExtraGroup variable in ParseWineTestReport().

This makes it clearer that it has nothing to do with setting up the
current group of errors.

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

---

 testbot/lib/WineTestBot/LogUtils.pm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index 7a72958..2cb2ad2 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -738,10 +738,10 @@ sub ParseWineTestReport($$$)
   my $ExtraCount = @{$LogInfo->{Extra}};
   if ($ExtraCount)
   {
-    my $CurGroup = _AddLogGroup($LogInfo, "Report validation errors");
-    $CurGroup->{Errors} = $LogInfo->{Extra};
+    my $ExtraGroup = _AddLogGroup($LogInfo, "Report validation errors");
+    $ExtraGroup->{Errors} = $LogInfo->{Extra};
     my @LineNos = (0) x $ExtraCount;
-    $CurGroup->{LineNos} = \@LineNos;
+    $ExtraGroup->{LineNos} = \@LineNos;
     $LogInfo->{ErrCount} += $ExtraCount;
   }
 




More information about the wine-cvs mailing list