Francois Gouget : testbot/WineSendLog: Use LogUtils to identify errors in the log files.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: 2ce3e4cf9d1ef1ce3314770f40a3a4037aaa06b3
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=2ce3e4cf9d1ef1ce3314770f40a3a4037aaa06b3

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Jun 26 13:29:55 2018 +0200

testbot/WineSendLog: Use LogUtils to identify errors in the log files.

GetLogLineCategory()'s error detection is much more complete and this
avoids code duplication.

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

---

 testbot/bin/WineSendLog.pl | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/testbot/bin/WineSendLog.pl b/testbot/bin/WineSendLog.pl
index 2f09779..3397809 100755
--- a/testbot/bin/WineSendLog.pl
+++ b/testbot/bin/WineSendLog.pl
@@ -45,6 +45,7 @@ use Algorithm::Diff;
 use WineTestBot::Config;
 use WineTestBot::Jobs;
 use WineTestBot::Log;
+use WineTestBot::LogUtils;
 use WineTestBot::StepsTasks;
 
 
@@ -290,8 +291,8 @@ EOF
         {
           $CurrentDll = $1;
         }
-        if ($Line =~ m/: Test failed: / || $Line =~ m/ done \(258\)/ ||
-            $Line =~ m/: unhandled exception [0-9a-fA-F]{8} at /)
+        my $Category = GetLogLineCategory($Line);
+        if ($Category eq "error")
         {
           if ($PrintedDll ne $CurrentDll)
           {