Francois Gouget : testbot: Simplify a loop in PatchUtils::GetPatchImpact() .

Alexandre Julliard julliard at winehq.org
Wed Jun 20 17:12:28 CDT 2018


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Wed Jun 20 02:30:35 2018 +0200

testbot: Simplify a loop in PatchUtils::GetPatchImpact().

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

---

 testbot/lib/WineTestBot/PatchUtils.pm | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/testbot/lib/WineTestBot/PatchUtils.pm b/testbot/lib/WineTestBot/PatchUtils.pm
index ec609d9..6eb8e3c 100644
--- a/testbot/lib/WineTestBot/PatchUtils.pm
+++ b/testbot/lib/WineTestBot/PatchUtils.pm
@@ -227,10 +227,8 @@ sub GetPatchImpact($;$$)
 
   $Impacts->{ModuleCount} = 0;
   $Impacts->{UnitCount} = 0;
-  foreach my $Module (keys %{$Impacts->{Tests}})
+  foreach my $TestInfo (values %{$Impacts->{Tests}})
   {
-    my $TestInfo = $Impacts->{Tests}->{$Module};
-
     # For each module, identify modifications to non-C files and helper dlls
     my $AllUnits;
     foreach my $File (keys %{$TestInfo->{Files}})




More information about the wine-cvs mailing list