Merge pull request #169 from arames/gh-pages

Fix the `cpplint.py` `build/endif_comment` check.
This commit is contained in:
Elliot Glaysher 2016-09-21 10:03:30 -07:00 committed by GitHub
commit 1342002613
2 changed files with 30 additions and 8 deletions

9
cpplint/cpplint.py vendored
View File

@ -2683,6 +2683,11 @@ def CheckForNonStandardConstructs(filename, clean_lines, linenum,
filename, line number, error level, and message 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. # Remove comments from the line, but leave in strings for now.
line = clean_lines.lines[linenum] line = clean_lines.lines[linenum]
@ -2713,10 +2718,6 @@ def CheckForNonStandardConstructs(filename, clean_lines, linenum,
'Storage-class specifier (static, extern, typedef, etc) should be ' 'Storage-class specifier (static, extern, typedef, etc) should be '
'at the beginning of the declaration.') '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): if Match(r'\s*class\s+(\w+\s*::\s*)+\w+\s*;', line):
error(filename, linenum, 'build/forward_decl', 5, error(filename, linenum, 'build/forward_decl', 5,
'Inner-style forward declarations are invalid. Remove this line.') 'Inner-style forward declarations are invalid. Remove this line.')

View File

@ -3754,7 +3754,7 @@ class CpplintTest(CpplintTestBase):
#else #else
baz; baz;
qux; qux;
#endif""", #endif // foo""",
'') '')
self.TestMultiLineLint( self.TestMultiLineLint(
"""void F() { """void F() {
@ -3910,7 +3910,7 @@ class CpplintTest(CpplintTestBase):
'#include "path/unique.h"', '#include "path/unique.h"',
'#else', '#else',
'#include "path/unique.h"', '#include "path/unique.h"',
'#endif', '#endif // MACRO',
''], ''],
error_collector) error_collector)
self.assertEquals( self.assertEquals(
@ -3959,7 +3959,7 @@ class CpplintTest(CpplintTestBase):
struct Foo : public Goo { struct Foo : public Goo {
#else #else
struct Foo : public Hoo { struct Foo : public Hoo {
#endif #endif // DERIVE_FROM_GOO
};""", };""",
'') '')
self.TestMultiLineLint( self.TestMultiLineLint(
@ -3969,7 +3969,7 @@ class CpplintTest(CpplintTestBase):
: public Goo { : public Goo {
#else #else
: public Hoo { : public Hoo {
#endif #endif // DERIVE_FROM_GOO
};""", };""",
'') '')
# Test incomplete class # Test incomplete class
@ -3987,6 +3987,27 @@ class CpplintTest(CpplintTestBase):
'Uncommented text after #endif is non-standard. Use a comment.' 'Uncommented text after #endif is non-standard. Use a comment.'
' [build/endif_comment] [5]') ' [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): def testBuildForwardDecl(self):
# The crosstool compiler we currently use will fail to compile the # The crosstool compiler we currently use will fail to compile the
# code in this test, so we might consider removing the lint check. # code in this test, so we might consider removing the lint check.