mirror of
https://github.com/google/styleguide.git
synced 2024-03-22 13:11:43 +08:00
Add support for c++17 tuple destructuring (#487)
C++17 adds support for tuple destructuring. This allow one to write code such as: ``` std::pair<int, int> span = getSpan(); auto [start, end] = span; // Use start as span.first and end as span.second ``` This makes cpplint recognize and allow a space before the '[' in this situation. This is a purposeful divergence from the internal version where the entire whitespace/braces category was removed. It was decided to leave the checks in since this is sometimes used without other formatting tools. Test: manual
This commit is contained in:
parent
83a9e8d7ca
commit
26470f9ccb
4
cpplint/cpplint.py
vendored
4
cpplint/cpplint.py
vendored
|
@ -3283,8 +3283,8 @@ def CheckSpacing(filename, clean_lines, linenum, nesting_state, error):
|
|||
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):
|
||||
# 'delete []', 'return []() {};', or 'auto [abc, ...] = ...;'.
|
||||
if Search(r'\w\s+\[', line) and not Search(r'(?:auto&?|delete|return)\s+\[', line):
|
||||
error(filename, linenum, 'whitespace/braces', 5,
|
||||
'Extra space before [')
|
||||
|
||||
|
|
Loading…
Reference in New Issue
Block a user