diff options
author | Etienne Bergeron <etienneb@google.com> | 2016-04-21 17:28:08 +0000 |
---|---|---|
committer | Etienne Bergeron <etienneb@google.com> | 2016-04-21 17:28:08 +0000 |
commit | 1dbd582387a577ee416d22d9cc2dc957926f8035 (patch) | |
tree | ebe510701c347d172d12c7c2b2ef76502fda4e87 /clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp | |
parent | a7a55c4d276523fecd460c503be885348caae502 (diff) | |
download | bcm5719-llvm-1dbd582387a577ee416d22d9cc2dc957926f8035.tar.gz bcm5719-llvm-1dbd582387a577ee416d22d9cc2dc957926f8035.zip |
[clang-tidy] New checker to detect suspicious string constructor.
Summary:
Checker to validate string constructor parameters.
A common mistake is to swap parameter for the fill-constructor.
```
std::string str('x', 4);
std::string str('4', x);
```
Reviewers: alexfh
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D19146
llvm-svn: 267011
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 aaeddeded21..da6706b94e3 100644 --- a/clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp +++ b/clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp @@ -33,6 +33,7 @@ #include "SizeofContainerCheck.h" #include "SizeofExpressionCheck.h" #include "StaticAssertCheck.h" +#include "StringConstructorCheck.h" #include "StringIntegerAssignmentCheck.h" #include "StringLiteralWithEmbeddedNulCheck.h" #include "SuspiciousMissingCommaCheck.h" @@ -99,6 +100,8 @@ public: "misc-sizeof-expression"); CheckFactories.registerCheck<StaticAssertCheck>( "misc-static-assert"); + CheckFactories.registerCheck<StringConstructorCheck>( + "misc-string-constructor"); CheckFactories.registerCheck<StringIntegerAssignmentCheck>( "misc-string-integer-assignment"); CheckFactories.registerCheck<StringLiteralWithEmbeddedNulCheck>( |