cpplint: restore runtime/sizeof
dropped in r113; this check is helpful in this project
Change-Id: If810607b7cbf89ad701b4865d685f4b96aeee2dc
diff --git a/tools/cpplint.py b/tools/cpplint.py
index 76d0735..25fbef7 100755
--- a/tools/cpplint.py
+++ b/tools/cpplint.py
@@ -181,6 +181,7 @@
'runtime/printf',
'runtime/printf_format',
'runtime/references',
+ 'runtime/sizeof',
'runtime/string',
'runtime/threadsafe_fn',
'runtime/vlog',
@@ -4154,10 +4155,12 @@
if not match:
return False
- # Exclude lines with sizeof, since sizeof looks like a cast.
+ # e.g., sizeof(int)
sizeof_match = Match(r'.*sizeof\s*$', line[0:match.start(1) - 1])
if sizeof_match:
- return False
+ error(filename, linenum, 'runtime/sizeof', 1,
+ 'Using sizeof(type). Use sizeof(varname) instead if possible')
+ return True
# operator++(int) and operator--(int)
if (line[0:match.start(1) - 1].endswith(' operator++') or
