[FIX] Bug: NOLINT, NOLINTNEXTLINE has no effect for "};".

suryalokhande · Jun 30, 2016 · b7e2ef6 · b7e2ef6
1 parent 70d6b7d
commit b7e2ef6
Showing 1 changed file with 8 additions and 0 deletions.
diff --git 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 }")