[PATCH] D60281: [analyzer] Add docs for cplusplus.InnerPointer
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 14 16:34:02 PDT 2019
NoQ added inline comments.
================
Comment at: clang/docs/analyzer/checkers.rst:263
+
+ void _deref_after_equals() {
+ std::string s = "llvm";
----------------
I suggest `deref_after_assignment`.
================
Comment at: clang/docs/analyzer/checkers.rst:271
+ const char *return_temp() {
+ int x;
+ return std::to_string(x).c_str(); // warn: inner pointer of container used after re/deallocation
----------------
Let's turn this into a parameter so that to avoid an uninitialized use (and generally be more realistic).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60281/new/
https://reviews.llvm.org/D60281
More information about the cfe-commits
mailing list