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

Francois Gouget fgouget at codeweavers.com
Tue Oct 24 12:04:55 CDT 2017


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

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 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 f6c19595..f42b88d7 100644
--- a/testbot/lib/WineTestBot/Jobs.pm
+++ b/testbot/lib/WineTestBot/Jobs.pm
@@ -400,6 +400,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;
 }
 
-- 
2.14.2



More information about the wine-patches mailing list