[clang-tools-extra] r247580 - [clang-tidy] updated misc-sizeof-container docs.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 14 09:56:58 PDT 2015


Author: alexfh
Date: Mon Sep 14 11:56:57 2015
New Revision: 247580

URL: http://llvm.org/viewvc/llvm-project?rev=247580&view=rev
Log:
[clang-tidy] updated misc-sizeof-container docs.

Modified:
    clang-tools-extra/trunk/docs/clang-tidy/checks/misc-sizeof-container.rst

Modified: clang-tools-extra/trunk/docs/clang-tidy/checks/misc-sizeof-container.rst
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/misc-sizeof-container.rst?rev=247580&r1=247579&r2=247580&view=diff
==============================================================================
--- clang-tools-extra/trunk/docs/clang-tidy/checks/misc-sizeof-container.rst (original)
+++ clang-tools-extra/trunk/docs/clang-tidy/checks/misc-sizeof-container.rst Mon Sep 14 11:56:57 2015
@@ -4,7 +4,9 @@ misc-sizeof-container
 The check finds usages of ``sizeof`` on expressions of STL container types. Most
 likely the user wanted to use ``.size()`` instead.
 
-Currently only ``std::string`` and ``std::vector<T>`` are supported.
+All class/struct types declared in namespace ``std::`` having a const ``size()``
+method are considered containers, with the exception of ``std::bitset`` and
+``std::array``.
 
 Examples:
 
@@ -12,9 +14,12 @@ Examples:
 
   std::string s;
   int a = 47 + sizeof(s); // warning: sizeof() doesn't return the size of the container. Did you mean .size()?
-                          // The suggested fix is: int a = 47 + s.size();
 
   int b = sizeof(std::string); // no warning, probably intended.
 
   std::string array_of_strings[10];
   int c = sizeof(array_of_strings) / sizeof(array_of_strings[0]); // no warning, definitely intended.
+
+  std::array<int, 3> std_array;
+  int d = sizeof(std_array); // no warning, probably intended.
+




More information about the cfe-commits mailing list