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