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

Francois Gouget fgouget at codeweavers.com
Tue Dec 17 04:01:42 CST 2013


---
 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;
-- 
1.8.5.1



More information about the wine-patches mailing list