diff options
author | Marco Hinz <mh.codebro@gmail.com> | 2022-05-16 00:14:31 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-05-16 00:14:31 +0200 |
commit | 59ba66d1ab64be1b01154d0cab7f8af9fe02e93d (patch) | |
tree | 6d5b44d2e90016f1849be7f17f150c643a35f217 /src/clint.py | |
parent | 5e3b16836a333d48f8977fc201c5e666767d61f1 (diff) | |
download | rneovim-59ba66d1ab64be1b01154d0cab7f8af9fe02e93d.tar.gz rneovim-59ba66d1ab64be1b01154d0cab7f8af9fe02e93d.tar.bz2 rneovim-59ba66d1ab64be1b01154d0cab7f8af9fe02e93d.zip |
ci(clint): remove checks for whitespace after comma/semicolon (#18584)
uncrustify is the source of truth where possible.
Remove any redundant checks from clint.py.
See also https://github.com/neovim/neovim/pull/18563
Diffstat (limited to 'src/clint.py')
-rwxr-xr-x | src/clint.py | 38 |
1 files changed, 0 insertions, 38 deletions
diff --git a/src/clint.py b/src/clint.py index 94414ef2da..b0992fcd63 100755 --- a/src/clint.py +++ b/src/clint.py @@ -199,7 +199,6 @@ _ERROR_CATEGORIES = [ 'whitespace/alignment', 'whitespace/blank_line', 'whitespace/braces', - 'whitespace/comma', 'whitespace/comments', 'whitespace/empty_conditional_body', 'whitespace/empty_loop_body', @@ -209,7 +208,6 @@ _ERROR_CATEGORIES = [ 'whitespace/newline', 'whitespace/operators', 'whitespace/parens', - 'whitespace/semicolon', 'whitespace/tab', 'whitespace/todo', 'whitespace/line_continuation', @@ -2362,29 +2360,6 @@ def CheckSpacing(filename, clean_lines, linenum, error): 'Should have zero or one spaces inside ( and ) in %s' % match.group(1)) - # You should always have a space after a comma (either as fn arg or - # operator). - # - # This does not apply when the non-space character following the - # comma is another comma, since the only time when that happens is - # for empty macro arguments. - # - # We run this check in two passes: first pass on elided lines to - # verify that lines contain missing whitespaces, second pass on raw - # lines to confirm that those missing whitespaces are not due to - # elided comments. - if Search(r',[^,\s]', line) and Search(r',[^,\s]', raw[linenum]): - error(filename, linenum, 'whitespace/comma', 3, - 'Missing space after ,') - - # You should always have a space after a semicolon - # except for few corner cases - # TODO(unknown): clarify if 'if (1) { return 1;}' is requires one more - # space after ; - if Search(r';[^\s};\\)/]', line): - error(filename, linenum, 'whitespace/semicolon', 3, - 'Missing space after ;') - # Next we will look for issues with function calls. CheckSpacingForFunctionCall(filename, line, linenum, error) @@ -2447,19 +2422,6 @@ def CheckSpacing(filename, clean_lines, linenum, error): error(filename, linenum, 'whitespace/braces', 5, 'Extra space before [') - # You shouldn't have a space before a semicolon at the end of the line. - if Search(r':\s*;\s*$', line): - error(filename, linenum, 'whitespace/semicolon', 5, - 'Semicolon defining empty statement. Use {} instead.') - elif Search(r'^\s*;\s*$', line): - error(filename, linenum, 'whitespace/semicolon', 5, - 'Line contains only semicolon. If this should be an empty' - ' statement, use {} instead.') - elif Search(r'\s+;\s*$', line): - error(filename, linenum, 'whitespace/semicolon', 5, - 'Extra space before last semicolon. If this should be an empty ' - 'statement, use {} instead.') - if Search(r'\{(?!\})\S', line): error(filename, linenum, 'whitespace/braces', 5, 'Missing space after {') |