Mercurial > public > mercurial-scm > hg-stable
comparison contrib/check-code.py @ 11672:dad185761392
check-code: add warning on lines over 80 characters
| author | Matt Mackall <mpm@selenic.com> |
|---|---|
| date | Sun, 25 Jul 2010 17:10:32 -0500 |
| parents | c5d40818b270 |
| children | 16723af520b0 |
comparison
equal
deleted
inserted
replaced
| 11671:ca6ede0988d5 | 11672:dad185761392 |
|---|---|
| 80 (r'\S;\s*\n', "semicolon"), | 80 (r'\S;\s*\n', "semicolon"), |
| 81 (r'\w,\w', "missing whitespace after ,"), | 81 (r'\w,\w', "missing whitespace after ,"), |
| 82 (r'\w[+/*\-<>]\w', "missing whitespace in expression"), | 82 (r'\w[+/*\-<>]\w', "missing whitespace in expression"), |
| 83 (r'^\s+\w+=\w+[^,)]$', "missing whitespace in assignment"), | 83 (r'^\s+\w+=\w+[^,)]$', "missing whitespace in assignment"), |
| 84 (r'.{85}', "line too long"), | 84 (r'.{85}', "line too long"), |
| 85 (r'.{81}', "warning: line over 80 characters"), | |
| 85 (r'[^\n]\Z', "no trailing newline"), | 86 (r'[^\n]\Z', "no trailing newline"), |
| 86 # (r'^\s+[^_ ][^_. ]+_[^_]+\s*=', "don't use underbars in identifiers"), | 87 # (r'^\s+[^_ ][^_. ]+_[^_]+\s*=', "don't use underbars in identifiers"), |
| 87 # (r'\w*[a-z][A-Z]\w*\s*=', "don't use camelcase in identifiers"), | 88 # (r'\w*[a-z][A-Z]\w*\s*=', "don't use camelcase in identifiers"), |
| 88 (r'^\s*(if|while|def|class|except|try)\s[^[]*:\s*[^\]#\s]+', | 89 (r'^\s*(if|while|def|class|except|try)\s[^[]*:\s*[^\]#\s]+', |
| 89 "linebreak after :"), | 90 "linebreak after :"), |