From f2ced939ebbe0fb491f0d306f01b7353c6896bd8 Mon Sep 17 00:00:00 2001 From: Piotr Semenov Date: Fri, 20 May 2016 18:39:34 +0300 Subject: [PATCH] [FIX] Bug: NOLINT, NOLINTNEXTLINE has no effect if used for the closing-brace line "};" in the lambda declaration. --- cpplint/cpplint.py | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/cpplint/cpplint.py b/cpplint/cpplint.py index 9491320..079f61d 100755 --- a/cpplint/cpplint.py +++ b/cpplint/cpplint.py @@ -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 }")