diff options
| author | maruel@chromium.org <maruel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-29 17:25:34 +0000 |
|---|---|---|
| committer | maruel@chromium.org <maruel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-29 17:25:34 +0000 |
| commit | cb155a802bdc49cfafea1c41e0f0a92168f1da09 (patch) | |
| tree | bf0fae4855c7f4458d8e959455b9d12d6cd54d9d /tools/valgrind/memcheck_analyze.py | |
| parent | 9bec23957253b83955c4096903ad6293d1c6e1bb (diff) | |
| download | chromium_src-cb155a802bdc49cfafea1c41e0f0a92168f1da09.zip chromium_src-cb155a802bdc49cfafea1c41e0f0a92168f1da09.tar.gz chromium_src-cb155a802bdc49cfafea1c41e0f0a92168f1da09.tar.bz2 | |
Fix python scripts in src/tools/
Make sure that:
- shebang is only present for executable files
- shebang is #!/usr/bin/env python
- __main__ is only present for executable files
- file's executable bit is coherent
Also fix EOF LF to be only one.
TBR=timurrrr@chromium.org
BUG=105108
TEST=
Review URL: http://codereview.chromium.org/8678023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111960 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/valgrind/memcheck_analyze.py')
| -rwxr-xr-x | tools/valgrind/memcheck_analyze.py | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/tools/valgrind/memcheck_analyze.py b/tools/valgrind/memcheck_analyze.py index e73ad26..b07c5d7 100755 --- a/tools/valgrind/memcheck_analyze.py +++ b/tools/valgrind/memcheck_analyze.py @@ -1,4 +1,4 @@ -#!/usr/bin/python +#!/usr/bin/env python # Copyright (c) 2011 The Chromium Authors. All rights reserved. # Use of this source code is governed by a BSD-style license that can be # found in the LICENSE file. @@ -603,9 +603,9 @@ class MemcheckAnalyzer: logging.info("PASS! No errors found!") return 0 + def _main(): '''For testing only. The MemcheckAnalyzer class should be imported instead.''' - retcode = 0 parser = optparse.OptionParser("usage: %prog [options] <files to analyze>") parser.add_option("", "--source_dir", help="path to top of source tree for this build" @@ -617,9 +617,8 @@ def _main(): filenames = args analyzer = MemcheckAnalyzer(options.source_dir, use_gdb=True) - retcode = analyzer.Report(filenames, None) + return analyzer.Report(filenames, None) - sys.exit(retcode) if __name__ == "__main__": - _main() + sys.exit(_main()) |