Francois Gouget : testbot: Renamed FilterHypervisors() to make it clearer it takes a list of hypervisors.

Alexandre Julliard julliard at winehq.org
Tue Dec 17 10:10:03 CST 2013


Module: tools
Branch: master
Commit: 4561043d9d57b397856838abc0c89bc4761ba2f2
URL:    http://source.winehq.org/git/tools.git/?a=commit;h=4561043d9d57b397856838abc0c89bc4761ba2f2

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Dec 17 11:01:42 2013 +0100

testbot: Renamed FilterHypervisors() to make it clearer it takes a list of hypervisors.

---

 testbot/lib/WineTestBot/Jobs.pm |    2 +-
 testbot/lib/WineTestBot/VMs.pm  |    6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm
index 94eae04..5d50fde 100644
--- a/testbot/lib/WineTestBot/Jobs.pm
+++ b/testbot/lib/WineTestBot/Jobs.pm
@@ -433,7 +433,7 @@ sub ScheduleOnHost($$)
 
   my $HostVMs = CreateVMs();
   $HostVMs->FilterEnabledRole();
-  $HostVMs->FilterHypervisor($Hypervisors);
+  $HostVMs->FilterHypervisors($Hypervisors);
 
   # Count the VMs that are 'active', that is, that use resources on the host,
   # and those that are reverting. Also build a prioritized list of those that
diff --git a/testbot/lib/WineTestBot/VMs.pm b/testbot/lib/WineTestBot/VMs.pm
index d4267e0..55c7c6c 100644
--- a/testbot/lib/WineTestBot/VMs.pm
+++ b/testbot/lib/WineTestBot/VMs.pm
@@ -702,12 +702,12 @@ sub FilterEnabledStatus($)
   $self->AddFilter("Status", ["dirty", "reverting", "sleeping", "idle", "running", "off"]);
 }
 
-sub FilterHypervisor
+sub FilterHypervisors($$)
 {
   my $self = shift;
-  my $Hypervisor = $_[0];
+  my $Hypervisors = $_[0];
 
-  $self->AddFilter("VirtURI", $Hypervisor);
+  $self->AddFilter("VirtURI", $Hypervisors);
 }
 
 1;




More information about the wine-cvs mailing list