diff options
| author | Alexander Kornienko <alexfh@google.com> | 2015-09-10 16:37:46 +0000 |
|---|---|---|
| committer | Alexander Kornienko <alexfh@google.com> | 2015-09-10 16:37:46 +0000 |
| commit | 7532d3e93d3980d628f10c63c2bc97bca2e7c2c1 (patch) | |
| tree | 39a8b5ff46f69b31cdaacdf4b95512a9a63bd93c /clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp | |
| parent | e3b1f2b7658ea22b13d6bae34f5ac4c9f4dc8fc6 (diff) | |
| download | llvm-7532d3e93d3980d628f10c63c2bc97bca2e7c2c1.zip llvm-7532d3e93d3980d628f10c63c2bc97bca2e7c2c1.tar.gz llvm-7532d3e93d3980d628f10c63c2bc97bca2e7c2c1.tar.bz2 | |
[clang-tidy] Add misc-sizeof-container check to find sizeof() uses on stl
containers.
Summary:
sizeof(some_std_string) is likely to be an error. This check finds this
pattern and suggests using .size() instead.
Reviewers: djasper, klimek, aaron.ballman
Subscribers: aaron.ballman, cfe-commits
Differential Revision: http://reviews.llvm.org/D12759
llvm-svn: 247297
Diffstat (limited to 'clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp')
| -rw-r--r-- | clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp b/clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp index 9976131..dc9a4eb 100644 --- a/clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp +++ b/clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp @@ -20,6 +20,7 @@ #include "MacroRepeatedSideEffectsCheck.h" #include "MoveConstructorInitCheck.h" #include "NoexceptMoveConstructorCheck.h" +#include "SizeofContainerCheck.h" #include "StaticAssertCheck.h" #include "SwappedArgumentsCheck.h" #include "UndelegatedConstructor.h" @@ -54,6 +55,8 @@ public: "misc-move-constructor-init"); CheckFactories.registerCheck<NoexceptMoveConstructorCheck>( "misc-noexcept-move-constructor"); + CheckFactories.registerCheck<SizeofContainerCheck>( + "misc-sizeof-container"); CheckFactories.registerCheck<StaticAssertCheck>( "misc-static-assert"); CheckFactories.registerCheck<SwappedArgumentsCheck>( |