[tools] testbot/LogUtils: Don't modify ParseWineTestReport()'s timeout lines.

[Francois Gouget]

Changing the timeout lines for reporting prevented the parser from
recognizing them and lead to extraneous missing failure / todo / skip
errors.

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
---
 testbot/lib/WineTestBot/LogUtils.pm | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index 863cca6bf4..1a87f174b7 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -532,17 +532,18 @@ sub ParseWineTestReport($$$)
     if (GetReportLineCategory($Line) eq "error")
     {
       # Make the timeout messages more user-friendly
-      if ($Line =~ m/^[^:]+:([^:]*):[0-9a-f]+ done \(258\)/)
+      my $ErrLine = $Line;
+      if ($ErrLine =~ /^[^:]+:([^:]*):[0-9a-f]+ done \(258\)/)
       {
         my $Unit = $1;
-        $Line = $Unit ne "" ? "$Unit: Timeout" : "Timeout";
+        $ErrLine = $Unit ne "" ? "$Unit: Timeout" : "Timeout";
       }
 
       if (!$CurGroup)
       {
         $CurGroup = _AddLogGroup($LogInfo, $CurGroupName, $CurGroupLineNo || $LineNo);
       }
-      _AddLogError($LogInfo, $CurGroup, $Line, $LineNo);
+      _AddLogError($LogInfo, $CurGroup, $ErrLine, $LineNo);
     }
 
     if ($Line =~ m%^([_.a-z0-9-]+):([_a-z0-9]*) (start|skipped) (?:-|[/_.a-z0-9-]+) (?:-|[.0-9a-f]+)\r?$%)
-- 
2.20.1