Francois Gouget : testbot: Document CompareJobPriority(), particularly the Priority field.

Alexandre Julliard julliard at winehq.org
Wed Oct 25 02:52:20 CDT 2017


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Oct 24 19:04:55 2017 +0200

testbot: Document CompareJobPriority(), particularly the Priority field.

Its naming is quite unfortunate since higher values mean a lower
priority.

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

---

 testbot/lib/WineTestBot/Jobs.pm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm
index 3c7ce0d..c787e8a 100644
--- a/testbot/lib/WineTestBot/Jobs.pm
+++ b/testbot/lib/WineTestBot/Jobs.pm
@@ -399,6 +399,8 @@ sub CreateJobs(;$)
 
 sub CompareJobPriority
 {
+  # Process Jobs with a higher Priority value last (it's a niceness in fact),
+  # and older Jobs first.
   return $a->Priority <=> $b->Priority || $a->Id <=> $b->Id;
 }
 




More information about the wine-cvs mailing list