Only strip the newline character at the end of the lines that we're considering

for length and for trailing whitespace; otherwise, the whitespace themselves
will also be removed.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@65182 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Misha Brukman 2009-02-20 22:28:45 +00:00
parent a2769a33c9
commit 4c4c952543

View File

@ -15,7 +15,7 @@ def VerifyLineLength(filename, lines, max_length):
""" """
line_num = 1 line_num = 1
for line in lines: for line in lines:
length = len(line.rstrip()) length = len(line.rstrip('\n'))
if length > max_length: if length > max_length:
print '%s:%d:Line exceeds %d chars (%d)' % (filename, line_num, print '%s:%d:Line exceeds %d chars (%d)' % (filename, line_num,
max_length, length) max_length, length)
@ -32,7 +32,7 @@ def VerifyTrailingWhitespace(filename, lines):
trailing_whitespace_re = re.compile(r'\s+$') trailing_whitespace_re = re.compile(r'\s+$')
line_num = 1 line_num = 1
for line in lines: for line in lines:
if trailing_whitespace_re.match(line.rstrip()): if trailing_whitespace_re.match(line.rstrip('\n')):
print '%s:%d:Trailing whitespace' % (filename, line_num) print '%s:%d:Trailing whitespace' % (filename, line_num)
line_num += 1 line_num += 1