[PATCH] Added option to clang-format-diff.py to just output diff without changing files.

Daniel Jasper djasper at google.com
Fri Oct 4 13:02:34 PDT 2013



================
Comment at: tools/clang-format/clang-format-diff.py:40
@@ -36,1 +39,3 @@
                                    'Reformat changed lines in diff.')
+  parser.add_argument('-n', action='store_true', default=False,
+                      help='don\'t change files, instead display what diffs'
----------------
Why "-n" is there precedence in other tools? Otherwise, how about a named parameter? Maybe --dry_run?

Or maybe two parameters "--dry_run" (do not apply changes) and "-v" (display diffs)?


http://llvm-reviews.chandlerc.com/D1840



More information about the cfe-commits mailing list