diff options
| author | Hans Wennborg <hans@hanshq.net> | 2012-07-27 19:17:46 +0000 |
|---|---|---|
| committer | Hans Wennborg <hans@hanshq.net> | 2012-07-27 19:17:46 +0000 |
| commit | 4684778993c667246039b4664acbce59dc99440c (patch) | |
| tree | 58ef4c76c3fa9b9e318c27634a3eae94cdadd0fb /include/clang/Analysis/Analyses | |
| parent | e61354b274ec5aa6acf3d15271896ce7596bb123 (diff) | |
Make -Wformat walk the typedef chain when looking for size_t, etc.
Clang's -Wformat fix-its currently suggest using "%zu" for values of
type size_t (in C99 or C++11 mode). However, for a type such as
std::vector<T>::size_type, it does not notice that type is actually
typedeffed to size_t, and instead suggests a format for the underlying
type, such as "%lu" or "%u".
This commit makes the format string fix mechanism walk the typedef chain
so that it notices if the type is size_t, even if that isn't "at the
top".
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@160886 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include/clang/Analysis/Analyses')
| -rw-r--r-- | include/clang/Analysis/Analyses/FormatString.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/include/clang/Analysis/Analyses/FormatString.h b/include/clang/Analysis/Analyses/FormatString.h index f99b97a189..1fb05d7486 100644 --- a/include/clang/Analysis/Analyses/FormatString.h +++ b/include/clang/Analysis/Analyses/FormatString.h @@ -355,6 +355,10 @@ public: bool hasStandardConversionSpecifier(const LangOptions &LangOpt) const; bool hasStandardLengthConversionCombination() const; + + /// For a TypedefType QT, if it is a named integer type such as size_t, + /// assign the appropriate value to LM and return true. + static bool namedTypeToLengthModifier(QualType QT, LengthModifier &LM); }; } // end analyze_format_string namespace |