Commit bd8c951
tools: disable readability/function cpplint rule
cpplint gets too easily confused by C++ constructs that look like
function declarations but aren't. Furthermore, it's arguably a
bad rule that conflicts with gcc's -Wunused-parameter flag.
PR-URL: #7334
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>1 parent cd38401 commit bd8c951
1 file changed
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -215,7 +215,11 @@ | |||
| 215 | 215 | # flag. By default all errors are on, so only add here categories that should be | |
| 216 | 216 | # off by default (i.e., categories that must be enabled by the --filter= flags). | |
| 217 | 217 | # All entries here should start with a '-' or '+', as in the --filter= flag. | |
| 218 | - _DEFAULT_FILTERS = [ '-build/include_alpha', '-legal/copyright' ] | ||
| 218 | + _DEFAULT_FILTERS = [ | ||
| 219 | + '-build/include_alpha', | ||
| 220 | + '-legal/copyright', | ||
| 221 | + '-readability/function', | ||
| 222 | + ] | ||
| 219 | 223 | ||
| 220 | 224 | # We used to check for high-bit characters, but after much discussion we | |
| 221 | 225 | # decided those were OK, as long as they were in UTF-8 and didn't represent | |
0 commit comments