diff options
author | timurrrr@chromium.org <timurrrr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-13 09:37:48 +0000 |
---|---|---|
committer | timurrrr@chromium.org <timurrrr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-13 09:37:48 +0000 |
commit | 425ca8da6cb8300481aa82f99d26dedd5baf1076 (patch) | |
tree | 86dec09a58408f06f1c50fb21d86fc2086895ef4 /tools | |
parent | 285ee614c774047f0fafd01bb717b125c3886098 (diff) | |
download | chromium_src-425ca8da6cb8300481aa82f99d26dedd5baf1076.zip chromium_src-425ca8da6cb8300481aa82f99d26dedd5baf1076.tar.gz chromium_src-425ca8da6cb8300481aa82f99d26dedd5baf1076.tar.bz2 |
Fix parsing of the first race report for ThreadSanitizer
We've introduced a mistake recently which results in hiding the first race report.
The reason was the "main thread" line which doesn't contain "{{{" and doesn't have context printed.
Please note that after submitting of this CL ThreadSanitizer bots can become red for a while.
Review URL: http://codereview.chromium.org/272026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28806 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools')
-rw-r--r-- | tools/valgrind/tsan_analyze.py | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/tools/valgrind/tsan_analyze.py b/tools/valgrind/tsan_analyze.py index 7c7d142..dcb5060 100644 --- a/tools/valgrind/tsan_analyze.py +++ b/tools/valgrind/tsan_analyze.py @@ -86,12 +86,13 @@ class TsanAnalyze: def ReadSection(self): result = [self.line_] - while not re.search('}}}', self.line_): - self.ReadLine() - if self.stack_trace_line_ is None: - result.append(self.line_) - else: - result.append(self.stack_trace_line_) + if re.search("{{{", self.line_): + while not re.search('}}}', self.line_): + self.ReadLine() + if self.stack_trace_line_ is None: + result.append(self.line_) + else: + result.append(self.stack_trace_line_) return result def ParseReportFile(self, filename): |