diff options
author | timurrrr@chromium.org <timurrrr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-13 17:13:12 +0000 |
---|---|---|
committer | timurrrr@chromium.org <timurrrr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-13 17:13:12 +0000 |
commit | 5e2f5a816c0821bed2b3bd8baa4fbfba70c86cd4 (patch) | |
tree | 3b386b9737b6c1c14df3aa5469e2bd81bf69971c /tools | |
parent | 655b2b1ae38dbcfe101cf0f0480818a5bc7ecf40 (diff) | |
download | chromium_src-5e2f5a816c0821bed2b3bd8baa4fbfba70c86cd4.zip chromium_src-5e2f5a816c0821bed2b3bd8baa4fbfba70c86cd4.tar.gz chromium_src-5e2f5a816c0821bed2b3bd8baa4fbfba70c86cd4.tar.bz2 |
Use the Valgrind and TSan format of printing the list of used suppressions
This should automatically work with http://chromium-build-logs.appspot.com/ once we move to the Memory FYI waterfall
TBR=bruening
TEST=tools\valgrind\chrome_tests.bat -t base --tool drmemory_light --gtest_filter="ToolsSanity.*"
Review URL: http://codereview.chromium.org/8277002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105318 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/valgrind/drmemory_analyze.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tools/valgrind/drmemory_analyze.py b/tools/valgrind/drmemory_analyze.py index 3a819fa..657a2e9 100755 --- a/tools/valgrind/drmemory_analyze.py +++ b/tools/valgrind/drmemory_analyze.py @@ -77,9 +77,9 @@ class DrMemoryAnalyze: if re.search("SUPPRESSIONS USED:", self.line_): self.ReadLine() while self.line_.strip() != "": - # TODO(timurrrr): use Valgrind format so the suppression dashboard - # counts used DrM suppressions too. - self.used_suppressions.append(self.line_.strip()) + line = self.line_.strip() + (count, name) = re.match(" *([0-9]+)x: (.*)", line).groups() + self.used_suppressions.append("%7s %s" % (count, name)) self.ReadLine() break @@ -93,8 +93,8 @@ class DrMemoryAnalyze: print "-----------------------------------------------------" # TODO(timurrrr): sum up the counts from different wrappers (e.g. ui_tests) # or does it work now already? Or add the memcheck-like per-test printing. - print "Suppressions used:\n count name\n ", # , adds ' ' instead of '\n' - print "\n ".join(self.used_suppressions) + print "Suppressions used:\n count name\n%s" % ( + "\n".join(self.used_suppressions)) print "-----------------------------------------------------" sys.stdout.flush() |