pseyfert created this revision.
pseyfert added reviewers: klimek, MyDeveloperDay.
pseyfert added projects: clang-format, clang.
The current clang-format.py does not handle trailing newlines at the end of a
file correctly.
Trailing empty lines get removed except one.
As far as I understand this is because clang-format gets fed from stdin and
writes to stdout when called from clang-format.py.
In a "normal" file (with no trailing empty lines) the string that gets passed
to clang-format does not contain a trailing '\n' after the '\n'.join from
python.
The clang-format binary does not add a trailing newline to input from stdin,
but (if there are multiple trailing '\n', all except one get removed).
When reading back this means that we see in python from a "normal" file a
string with no trailing '\n'. From a file with (potentially multiple) empty
line(s) at the end, we get a string with one trailing '\n' back in python. In
the former case all is fine, in the latter case split('\n') makes one empty
line at the end of the file out of the clang-format output. Desired would be
instead that the **file** ends with a newline, but not with an empty line.
For the case that a user specifies a range to format (and wants to keep
trailing empty lines) I did **not** try to fix this by simply removing all
trailing newlines from the clang-format output. Instead, I add a '\n' to the
unformatted file content (i.e. newline-terminate what is passed to
clang-format) and then strip off the last newline from the output (which itself
is now for sure the newline termination of the clang-format output).
(Should this get approved, I'll need someone to help me land this.)
https://reviews.llvm.org/D70864
Files:
clang/tools/clang-format/clang-format.py
Index: clang/tools/clang-format/clang-format.py
===================================================================
--- clang/tools/clang-format/clang-format.py
+++ clang/tools/clang-format/clang-format.py
@@ -70,7 +70,7 @@
# Get the current text.
encoding = vim.eval("&encoding")
buf = get_buffer(encoding)
- text = '\n'.join(buf)
+ text = '\n'.join(buf) + '\n'
# Determine range to format.
if vim.eval('exists("l:lines")') == '1':
@@ -129,7 +129,7 @@
else:
lines = stdout.decode(encoding).split('\n')
output = json.loads(lines[0])
- lines = lines[1:]
+ lines = lines[1:-1]
sequence = difflib.SequenceMatcher(None, buf, lines)
for op in reversed(sequence.get_opcodes()):
if op[0] is not 'equal':
Index: clang/tools/clang-format/clang-format.py
===================================================================
--- clang/tools/clang-format/clang-format.py
+++ clang/tools/clang-format/clang-format.py
@@ -70,7 +70,7 @@
# Get the current text.
encoding = vim.eval("&encoding")
buf = get_buffer(encoding)
- text = '\n'.join(buf)
+ text = '\n'.join(buf) + '\n'
# Determine range to format.
if vim.eval('exists("l:lines")') == '1':
@@ -129,7 +129,7 @@
else:
lines = stdout.decode(encoding).split('\n')
output = json.loads(lines[0])
- lines = lines[1:]
+ lines = lines[1:-1]
sequence = difflib.SequenceMatcher(None, buf, lines)
for op in reversed(sequence.get_opcodes()):
if op[0] is not 'equal':
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
