Francois Gouget : testbot: CompareTaskStatus() is not really needed. Remove it.
Alexandre Julliard
julliard at winehq.org
Tue Nov 14 09:51:42 CST 2017
Module: tools
Branch: master
Commit: da850add297378c3b21e354894873e10e7a1c953
URL: http://source.winehq.org/git/tools.git/?a=commit;h=da850add297378c3b21e354894873e10e7a1c953
Author: Francois Gouget <fgouget at codeweavers.com>
Date: Mon Nov 13 17:06:24 2017 +0100
testbot: CompareTaskStatus() is not really needed. Remove it.
The scheduler only cares about queued Tasks so there is no need to
compare their Status fields. So the comparison code can easily be
inlined like for Steps.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
testbot/lib/WineTestBot/Jobs.pm | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm
index c787e8a..c6eb290 100644
--- a/testbot/lib/WineTestBot/Jobs.pm
+++ b/testbot/lib/WineTestBot/Jobs.pm
@@ -404,11 +404,6 @@ sub CompareJobPriority
return $a->Priority <=> $b->Priority || $a->Id <=> $b->Id;
}
-sub CompareTaskStatus
-{
- return $b->Status cmp $a->Status || $a->No <=> $b->No;
-}
-
sub min(@)
{
my $m = shift @_;
@@ -543,7 +538,7 @@ sub ScheduleOnHost($$$)
my $PrepareNextStep;
my $Tasks = $Step->Tasks;
$Tasks->AddFilter("Status", ["queued"]);
- my @SortedTasks = sort CompareTaskStatus @{$Tasks->GetItems()};
+ my @SortedTasks = sort { $a->No <=> $b->No } @{$Tasks->GetItems()};
foreach my $Task (@SortedTasks)
{
my $VM = $Task->VM;
@@ -591,7 +586,7 @@ sub ScheduleOnHost($$$)
my $Step = $SortedSteps[1];
$Tasks = $Step->Tasks;
$Tasks->AddFilter("Status", ["queued"]);
- @SortedTasks = sort CompareTaskStatus @{$Tasks->GetItems()};
+ @SortedTasks = sort { $a->No <=> $b->No } @{$Tasks->GetItems()};
foreach my $Task (@SortedTasks)
{
my $VM = $Task->VM;
More information about the wine-cvs
mailing list