Skip to content

Commit

Permalink
[FIX] Bug: NOLINT, NOLINTNEXTLINE has no effect if used for the closi…
Browse files Browse the repository at this point in the history
…ng-brace line "};" in the lambda declaration.
  • Loading branch information
piotr-semenov committed May 20, 2016
1 parent 70d6b7d commit f2ced93
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions cpplint/cpplint.py
Original file line number Diff line number Diff line change
Expand Up @@ -3884,6 +3884,14 @@ def CheckTrailingSemicolon(filename, clean_lines, linenum, error):
# outputting warnings for the matching closing brace, if there are
# nested blocks with trailing semicolons, we will get the error
# messages in reversed order.

# We need to check the line forward for NOLINT
raw_lines = clean_lines.raw_lines
ParseNolintSuppressions(filename, raw_lines[endlinenum-1], endlinenum-1,
error)
ParseNolintSuppressions(filename, raw_lines[endlinenum], endlinenum,
error)

error(filename, endlinenum, 'readability/braces', 4,
"You don't need a ; after a }")

Expand Down

0 comments on commit f2ced93

Please sign in to comment.