summaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authortimurrrr@chromium.org <timurrrr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-10-13 17:13:12 +0000
committertimurrrr@chromium.org <timurrrr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-10-13 17:13:12 +0000
commit5e2f5a816c0821bed2b3bd8baa4fbfba70c86cd4 (patch)
tree3b386b9737b6c1c14df3aa5469e2bd81bf69971c /tools
parent655b2b1ae38dbcfe101cf0f0480818a5bc7ecf40 (diff)
downloadchromium_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-xtools/valgrind/drmemory_analyze.py10
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()