Skip to content

Commit aeb99e8

Browse files
author
Daniel Kroening
authored
Merge pull request #558 from rjmunro/feature/cpplint-sed-format
cpplint sed format improvements
2 parents 36c2e2d + 0b11c43 commit aeb99e8

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

scripts/cpplint.py

+6-2
Original file line numberDiff line numberDiff line change
@@ -550,9 +550,13 @@
550550
"Remove space before ( in switch (": "s/switch (/switch(/",
551551
"Should have a space between // and comment": 's/\/\//\/\/ /',
552552
"Missing space before {": r's/\([^ ]\){/\1 {/',
553-
"Tab found, replace by spaces": r's/\t/ /',
553+
"Tab found, replace by spaces": r's/\t/ /g',
554554
"Line ends in whitespace. Consider deleting these extra spaces.": r's/\s*$//',
555-
#"Redundant blank line at the end of a code block should be deleted.": "d", # messes up line numbers for other errors.
555+
"You don't need a ; after a }": r's/};/}/',
556+
"Missing space after ,": r's/,\([^ ]\)/, \1/g',
557+
# "Statement after an if should be on a new line": r's/^\(\s*\)if *\(([^()]*)\) *\(.*\)$/\1if\2\n\1 \3/', # Single layer of nested bracets
558+
# "Statement after an if should be on a new line": r's/^\(\s*\)if *\((\([^()]\|([^()]*)\)*)\) *\(.*\)$/\1if\2\n\1 \4/', # Max 2 layers of nested bracets; messes up line numbers for other errors.
559+
# "Redundant blank line at the end of a code block should be deleted.": "d", # messes up line numbers for other errors.
556560
}
557561

558562
_regexp_compile_cache = {}

0 commit comments

Comments
 (0)