diff options
author | Nathan Binkert <nate@binkert.org> | 2007-07-28 21:57:24 -0700 |
---|---|---|
committer | Nathan Binkert <nate@binkert.org> | 2007-07-28 21:57:24 -0700 |
commit | 398f6a70a788f1459cec7509859450984e442690 (patch) | |
tree | eef40ee564b6160b1540aa6b7db410b3eafaaa23 | |
parent | aaf59949e58ceb617aa4efd04597a63c88638b9d (diff) | |
parent | 41a1acd1e957d6856d5d5c0ddad3d39aa700c01d (diff) | |
download | gem5-398f6a70a788f1459cec7509859450984e442690.tar.xz |
merge: style.py fix
--HG--
extra : convert_revision : 2bb456f292a862e1615c370c1d14a3f709860c38
-rw-r--r-- | util/style.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/util/style.py b/util/style.py index 897617779..7238b9801 100644 --- a/util/style.py +++ b/util/style.py @@ -244,7 +244,7 @@ def validate(filename, stats, verbose, exit_code): msg(i, line, 'improper spacing after %s' % match.group(1)) bad() -def modified_lines(old_data, new_data): +def modified_lines(old_data, new_data, max_lines): from itertools import count from mercurial import bdiff, mdiff @@ -256,6 +256,8 @@ def modified_lines(old_data, new_data): modified.add(i) elif i + 1 >= fend: break + elif i > max_lines: + break return modified def check_whitespace(ui, repo, hooktype, node, parent1, parent2): @@ -301,13 +303,14 @@ def check_whitespace(ui, repo, hooktype, node, parent1, parent2): assert len(pctx) in (1, 2) file_data = fctx.data() - mod_lines = modified_lines(pctx[0].data(), file_data) + lines = mdiff.splitnewlines(file_data) + mod_lines = modified_lines(pctx[0].data(), file_data, len(lines)) if len(pctx) == 2: m2 = modified_lines(pctx[1].data(), file_data) mod_lines = mod_lines & m2 # only the lines that are new in both fixonly = set() - for i,line in enumerate(mdiff.splitnewlines(file_data)): + for i,line in enumerate(lines): if i not in mod_lines: continue |