Francois Gouget : testbot/LogUtils: Add support for winedbg's unhandled exception lines.

Alexandre Julliard julliard at winehq.org
Wed Feb 10 15:32:34 CST 2021


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Wed Feb 10 02:16:01 2021 +0100

testbot/LogUtils: Add support for winedbg's unhandled exception lines.

They replace the test framework's "unhandled exception" lines we get on
Windows but specify neither the test unit nor the process pid. So just
assign them to the current test unit.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=50538
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/lib/WineTestBot/LogUtils.pm | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index 69299da..9164b47 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -631,6 +631,13 @@ sub ParseWineTestReport($$$)
       _CheckUnit($LogInfo, $Cur, $Unit, "unhandled exception");
       $Cur->{LineFailures}++;
     }
+    elsif ($Line =~ /Unhandled exception: .* in .* code /)
+    {
+      # This also replaces a test summary line. The pid is unknown so use 0.
+      $Cur->{Pids}->{0} = 1;
+      $Cur->{SummaryFailures}++;
+      $Cur->{LineFailures}++;
+    }
     elsif (($Cur->{Unit} ne "" and
             $Line =~ /($Cur->{UnitsRE}):\d+: unhandled exception [0-9a-fA-F]{8} in child process /) or
            $Line =~ /^([_.a-z0-9]+):\d+: unhandled exception [0-9a-fA-F]{8} in child process /)




More information about the wine-cvs mailing list