Revert "Fix the cpplint.py build/endif_comment check."

eglaysher · tkruse · commit 3b83a5caebe8 · 2019-02-18T07:48:09.000+09:00
diff --git a/cpplint.py b/cpplint.py
@@ -3023,11 +3023,6 @@ def CheckForNonStandardConstructs(filename, clean_lines, linenum,
            filename, line number, error level, and message
   """
 
-  line = clean_lines.lines_without_raw_strings[linenum]
-  if Match(r'\s*#\s*endif\s*([^/\s]|/[^/]|$)', line):
-    error(filename, linenum, 'build/endif_comment', 5,
-          'Uncommented text after #endif is non-standard.  Use a comment.')
-
   # Remove comments from the line, but leave in strings for now.
   line = clean_lines.lines[linenum]
 
@@ -3058,6 +3053,10 @@ def CheckForNonStandardConstructs(filename, clean_lines, linenum,
           'Storage-class specifier (static, extern, typedef, etc) should be '
           'at the beginning of the declaration.')
 
+  if Match(r'\s*#\s*endif\s*[^/\s]+', line):
+    error(filename, linenum, 'build/endif_comment', 5,
+          'Uncommented text after #endif is non-standard.  Use a comment.')
+
   if Match(r'\s*class\s+(\w+\s*::\s*)+\w+\s*;', line):
     error(filename, linenum, 'build/forward_decl', 5,
           'Inner-style forward declarations are invalid.  Remove this line.')
diff --git a/cpplint_unittest.py b/cpplint_unittest.py
@@ -3936,7 +3936,7 @@ def testConditionals(self):
         #else
           baz;
           qux;
-        #endif  // foo""",
+        #endif""",
         '')
     self.TestMultiLineLint(
         """void F() {
@@ -4233,7 +4233,7 @@ def testDuplicateHeader(self):
                              '#include "path/unique.h"',
                              '#else',
                              '#include "path/unique.h"',
-                             '#endif  // MACRO',
+                             '#endif',
                              ''],
                             error_collector)
     self.assertEqual(
@@ -4284,7 +4284,7 @@ def testBuildClass(self):
           struct Foo : public Goo {
         #else
           struct Foo : public Hoo {
-        #endif  // DERIVE_FROM_GOO
+        #endif
           };""",
         '')
     self.TestMultiLineLint(
@@ -4294,7 +4294,7 @@ class Foo
           : public Goo {
         #else
           : public Hoo {
-        #endif  // DERIVE_FROM_GOO
+        #endif
         };""",
         '')
     # Test incomplete class
@@ -4312,27 +4312,6 @@ def testBuildEndComment(self):
         'Uncommented text after #endif is non-standard.  Use a comment.'
         '  [build/endif_comment] [5]')
 
-    correct_lines = [
-      '#endif  // text',
-      '#endif  //'
-    ]
-
-    for line in correct_lines:
-      self.TestLint(line, '')
-
-    incorrect_lines = [
-      '#endif',
-      '#endif Not a comment',
-      '#endif / One `/` is not enough to start a comment'
-    ]
-
-    for line in incorrect_lines:
-      self.TestLint(
-        line,
-        'Uncommented text after #endif is non-standard.  Use a comment.'
-        '  [build/endif_comment] [5]')
-
-
   def testBuildForwardDecl(self):
     # The crosstool compiler we currently use will fail to compile the
     # code in this test, so we might consider removing the lint check.
