StarMirror/styleguide
1
0
Fork 0
mirror of https://github.com/google/styleguide.git synced 2024-03-22 13:11:43 +08:00
Code Issues Projects Releases Wiki Activity

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

Browse Source
This commit is contained in:
Adrian 2016-12-27 19:50:41 -05:00 committed by GitHub
parent 159b4c81bb
commit 1c56a952a1
2 changed files with 30 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
cpplint/cpplint.py vendored
View File

@ -2710,6 +2710,11 @@ 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]
@ -2740,10 +2745,6 @@ 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.')

29
cpplint/cpplint_unittest.py
View File

@ -3754,7 +3754,7 @@ class CpplintTest(CpplintTestBase):
#else
baz;
qux;
#endif""",
#endif // foo""",
'')
self.TestMultiLineLint(
"""void F() {
@ -3920,7 +3920,7 @@ class CpplintTest(CpplintTestBase):
'#include "path/unique.h"',
'#else',
'#include "path/unique.h"',
'#endif',
'#endif // MACRO',
''],
error_collector)
self.assertEquals(
@ -3969,7 +3969,7 @@ class CpplintTest(CpplintTestBase):
struct Foo : public Goo {
#else
struct Foo : public Hoo {
#endif
#endif // DERIVE_FROM_GOO
};""",
'')
self.TestMultiLineLint(
@ -3979,7 +3979,7 @@ class CpplintTest(CpplintTestBase):
: public Goo {
#else
: public Hoo {
#endif
#endif // DERIVE_FROM_GOO
};""",
'')
# Test incomplete class
@ -3997,6 +3997,27 @@ class CpplintTest(CpplintTestBase):
'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.