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

Allow space before C++11 attributes

Browse Source 
Modify rule checking for spaces before `[`s to ignore C++11 attributes,
which usually should have spaces. This fixes an incorrect warning on
code like:

  #define ABIEXPORT [[gnu::visibility("default")]]
This commit is contained in:
Matthew Woehlke 2016-08-10 10:49:55 -04:00
parent f15e633de5
commit 27929f1fd6
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
cpplint/cpplint.py vendored
View File

@ -3188,9 +3188,10 @@ def CheckSpacing(filename, clean_lines, linenum, nesting_state, error):
# get rid of comments and strings
line = clean_lines.elided[linenum]
# You shouldn't have spaces before your brackets, except maybe after
# 'delete []' or 'return []() {};'
if Search(r'\w\s+\[', line) and not Search(r'(?:delete|return)\s+\[', line):
# You shouldn't have spaces before your brackets, except for C++11 attributes
# or maybe after 'delete []' or 'return []() {};'
if (Search(r'\w\s+\[(?!\[)', line) and
not Search(r'(?:delete|return)\s+\[', line)):
error(filename, linenum, 'whitespace/braces', 5,
'Extra space before [')