diff options
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/purify/common.py | 3 | ||||
| -rw-r--r-- | tools/purify/purify_test.py | 16 |
2 files changed, 16 insertions, 3 deletions
diff --git a/tools/purify/common.py b/tools/purify/common.py index a684bb6..d384514 100644 --- a/tools/purify/common.py +++ b/tools/purify/common.py @@ -52,8 +52,7 @@ class TimeoutError(Exception): pass def _print_line(line, flush=True): # Printing to a text file (including stdout) on Windows always winds up # using \r\n automatically. On buildbot, this winds up being read by a master - # running on Linux, so this is a pain. Unfortunately, it doesn't matter what - # we do here, so just leave this comment for future reference. + # running on Linux, so we manually convert crlf to '\n' print line.rstrip() + '\n', if flush: sys.stdout.flush() diff --git a/tools/purify/purify_test.py b/tools/purify/purify_test.py index 2d8f48a..c4798cef 100644 --- a/tools/purify/purify_test.py +++ b/tools/purify/purify_test.py @@ -100,11 +100,25 @@ class Purify(common.Rational): "[Purify]", "option -cache-dir=\"%s\"" % (self._cache_dir), "option -save-text-data=\"%s\"" % (common.FixPath(self._out_file)), + # Change the recorded stack depth to be much larger than the default. + # (webkit/v8 stacks in particular seem to get quite deep) "option -alloc-call-stack-length=30", "option -error-call-stack-length=30", "option -free-call-stack-length=30", + # Report leaks. "option -leaks-at-exit=yes", - "option -in-use-at-exit=no" + # Don't report memory in use (that's for memory profiling). + "option -in-use-at-exit=no", + # The maximum number of subprocesses. If this is exceeded, Purify + # seems to lose its mind, and we have a number of tests that use + # much larger than the default of 5. + "option -number-of-puts=30", + # With our large pdbs, purify's default timeout (30) isn't always + # enough. If this isn't enough, -1 means no timeout. + "option -server-comm-timeout=120", + # check stack memory loads for UMRs, etc. + # currently disabled due to noisiness (see bug 5189) + #"option -stack-load-checking=yes", ] ini_file = self._exe.replace(".exe", "_pure.ini") if os.path.isfile(ini_file): |