Francois Gouget : testbot/lib: Simplify a couple of job sorting functions.

Alexandre Julliard julliard at winehq.org
Mon Oct 29 12:19:47 CDT 2012


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Mon Oct 29 15:39:13 2012 +0100

testbot/lib: Simplify a couple of job sorting functions.

---

 testbot/lib/WineTestBot/Jobs.pm |   16 ++--------------
 1 files changed, 2 insertions(+), 14 deletions(-)

diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm
index b56e203..de66023 100644
--- a/testbot/lib/WineTestBot/Jobs.pm
+++ b/testbot/lib/WineTestBot/Jobs.pm
@@ -357,24 +357,12 @@ sub CreateJobs
 
 sub CompareJobPriority
 {
-  my $Compare = $a->Priority <=> $b->Priority;
-  if ($Compare == 0)
-  {
-    $Compare = $a->Id <=> $b->Id;
-  }
-
-  return $Compare;
+  return $a->Priority <=> $b->Priority || $a->Id <=> $b->Id;
 }
 
 sub CompareTaskStatus
 {
-  my $Compare = $b->Status cmp $a->Status;
-  if ($Compare == 0)
-  {
-    $Compare = $a->No <=> $b->No;
-  }
-
-  return $Compare;
+  return $b->Status cmp $a->Status || $a->No <=> $b->No;
 }
 
 =pod




More information about the wine-cvs mailing list