Changed clang-format-diff.py to output diff by default. Added -i option to apply changes to files instead.
Summary:
"svn diff|clang-format-diff.py" will just output the diff.
Now it's possible to use:
svn diff|clang-format-diff.py|patch -p0
as an equivalent to:
svn diff|clang-format-diff.py -i
;)
Reviewers: djasper
Reviewed By: djasper
CC: cfe-commits
Differential Revision: http://llvm-reviews.chandlerc.com/D1840
git-svn-id: svn://svn.chromium.org/llvm-project/cfe/trunk/tools/clang-format@192505 0b72dbe1-c17f-4bc7-b9db-2b4152be0356
diff --git a/clang-format-diff.py b/clang-format-diff.py
index ca949b8..60b8fb7 100755
--- a/clang-format-diff.py
+++ b/clang-format-diff.py
@@ -17,13 +17,16 @@
lines. This is useful to reformat all the lines touched by a specific patch.
Example usage for git users:
- git diff -U0 HEAD^ | clang-format-diff.py -p1
+ git diff -U0 HEAD^ | clang-format-diff.py -p1 -i
"""
import argparse
+import difflib
import re
+import string
import subprocess
+import StringIO
import sys
@@ -33,7 +36,11 @@
def main():
parser = argparse.ArgumentParser(description=
- 'Reformat changed lines in diff.')
+ 'Reformat changed lines in diff. Without -i '
+ 'option just output the diff that would be'
+ 'introduced.')
+ parser.add_argument('-i', action='store_true', default=False,
+ help='apply edits to files instead of displaying a diff')
parser.add_argument('-p', default=0,
help='strip the smallest prefix containing P slashes')
parser.add_argument(
@@ -71,7 +78,9 @@
# Reformat files containing changes in place.
for filename, lines in lines_by_file.iteritems():
- command = [binary, '-i', filename]
+ command = [binary, filename]
+ if args.i:
+ command.append('-i')
command.extend(lines)
if args.style:
command.extend(['-style', args.style])
@@ -84,6 +93,16 @@
if p.returncode != 0:
sys.exit(p.returncode);
+ if not args.i:
+ with open(filename) as f:
+ code = f.readlines()
+ formatted_code = StringIO.StringIO(stdout).readlines()
+ diff = difflib.unified_diff(code, formatted_code,
+ filename, filename,
+ '(before formatting)', '(after formatting)')
+ diff_string = string.join(diff, '')
+ if len(diff_string) > 0:
+ print diff_string
if __name__ == '__main__':
main()