]> granicus.if.org Git - libvpx/commitdiff
cpplint: restore runtime/sizeof
authorJames Zern <jzern@google.com>
Wed, 22 Jan 2014 01:56:04 +0000 (17:56 -0800)
committerJames Zern <jzern@google.com>
Wed, 22 Jan 2014 01:56:04 +0000 (17:56 -0800)
dropped in r113; this check is helpful in this project

Change-Id: If810607b7cbf89ad701b4865d685f4b96aeee2dc

tools/cpplint.py

index 76d073527832bba82b82f79e36e4916a874c76bf..25fbef73d8edac6526704db76dacd5b7f70b65df 100755 (executable)
@@ -181,6 +181,7 @@ _ERROR_CATEGORIES = [
   'runtime/printf',
   'runtime/printf_format',
   'runtime/references',
+  'runtime/sizeof',
   'runtime/string',
   'runtime/threadsafe_fn',
   'runtime/vlog',
@@ -4154,10 +4155,12 @@ def CheckCStyleCast(filename, linenum, line, raw_line, cast_type, pattern,
   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