From 76eec3db2173079ce103fbd3ac53892ddec8b963 Mon Sep 17 00:00:00 2001 From: "Jason R. Coombs" Date: Tue, 26 Jul 2011 11:38:04 -0400 Subject: Issue #10639: reindent.py tool now accepts a --newline option to specify the newline to be used in the output of converted files. --- Misc/NEWS | 1 + Tools/scripts/reindent.py | 16 ++++++++++++---- 2 files changed, 13 insertions(+), 4 deletions(-) diff --git a/Misc/NEWS b/Misc/NEWS index eb55f29..6010328 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -1075,6 +1075,7 @@ Tools/Demos - Issue #10639: reindent.py no longer converts newlines and will raise an error if attempting to convert a file with mixed newlines. + "--newline" option added to specify new line character. Extension Modules ----------------- diff --git a/Tools/scripts/reindent.py b/Tools/scripts/reindent.py index b18993b..ed66bc1 100755 --- a/Tools/scripts/reindent.py +++ b/Tools/scripts/reindent.py @@ -8,6 +8,8 @@ -r (--recurse) Recurse. Search for all .py files in subdirectories too. -n (--nobackup) No backup. Does not make a ".bak" file before reindenting. -v (--verbose) Verbose. Print informative msgs; else no output. + (--newline) Newline. Specify the newline character to use (CRLF, LF). + Default is the same as the original file. -h (--help) Help. Print this usage information and exit. Change Python (.py) files to use 4-space indents and no hard tab characters. @@ -65,10 +67,11 @@ def errprint(*args): def main(): import getopt - global verbose, recurse, dryrun, makebackup + global verbose, recurse, dryrun, makebackup, spec_newline + spec_newline = None try: opts, args = getopt.getopt(sys.argv[1:], "drnvh", - ["dryrun", "recurse", "nobackup", "verbose", "help"]) + ["dryrun", "recurse", "nobackup", "verbose", "newline=", "help"]) except getopt.error as msg: usage(msg) return @@ -81,6 +84,11 @@ def main(): makebackup = False elif o in ('-v', '--verbose'): verbose = True + elif o in ('--newline',): + if not a.upper() in ('CRLF', 'LF'): + usage() + return + spec_newline = dict(CRLF='\r\n', LF='\n')[a.upper()] elif o in ('-h', '--help'): usage() return @@ -118,9 +126,9 @@ def check(file): errprint("%s: I/O Error: %s" % (file, str(msg))) return - newline = r.newlines + newline = spec_newline if spec_newline else r.newlines if isinstance(newline, tuple): - errprint("%s: mixed newlines detected; cannot process file" % file) + errprint("%s: mixed newlines detected; cannot continue without --newline" % file) return if r.run(): -- cgit v0.12