max/lean4-htt
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

chore(cmake/Modules/cpplint): disable yet another buggy check

Browse source
This commit is contained in:
Leonardo de Moura 2018-06-06 15:28:44 -07:00
parent 0a5e7ff1a9
commit 744bca1964
1 changed files with 12 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
src/cmake/Modules/cpplint.py vendored
View file

@ -3312,13 +3312,18 @@ def CheckLanguage(filename, clean_lines, linenum, file_extension, include_state,
# are missing context is if MOCK_METHOD was split across
# multiple lines (for example http://go/hrfhr ), so we only need
# to check the previous line for MOCK_METHOD.
if (linenum == 0 or
not Match(r'^\s*MOCK_(CONST_)?METHOD\d+(_T)?\(\S+,\s*$',
clean_lines.elided[linenum - 1])):
error(filename, linenum, 'readability/casting', 4,
'Using deprecated casting style. '
'Use static_cast<%s>(...) instead' %
match.group(2))
pass
# Disabled by Leo de Moura, 06/06/2018.
# This test was producing bogus error messages when using std::function
# Example: optional<local_decl> local_ctx::find_if(std::function<bool(local_decl const &)> const & pred) const {
# if (linenum == 0 or
# not Match(r'^\s*MOCK_(CONST_)?METHOD\d+(_T)?\(\S+,\s*$',
# clean_lines.elided[linenum - 1])):
# error(filename, linenum, 'readability/casting', 4,
# 'Using deprecated casting style. '
# 'Use static_cast<%s>(...) instead' %
# match.group(2))
CheckCStyleCast(filename, linenum, line, clean_lines.raw_lines[linenum],
'static_cast',