This revision was automatically updated to reflect the committed changes.
Closed by commit rC336497: [analyzer] Add support for data() in
DanglingInternalBufferChecker. (authored by rkovacs, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D48532?vs=152737&id=154501#toc
Reposi
rnkovacs added inline comments.
Comment at: test/Analysis/dangling-internal-buffer.cpp:10
const CharT *c_str();
+ const CharT *data();
};
xazax.hun wrote:
> Note that these methods are const according to the standard. Even if it does
> not make any differen
rnkovacs updated this revision to Diff 152737.
rnkovacs marked an inline comment as done.
https://reviews.llvm.org/D48532
Files:
lib/StaticAnalyzer/Checkers/DanglingInternalBufferChecker.cpp
test/Analysis/dangling-internal-buffer.cpp
Index: test/Analysis/dangling-internal-buffer.cpp
xazax.hun added inline comments.
Comment at: test/Analysis/dangling-internal-buffer.cpp:10
const CharT *c_str();
+ const CharT *data();
};
Note that these methods are const according to the standard. Even if it does
not make any difference for this check we
xazax.hun accepted this revision.
xazax.hun added a comment.
This revision is now accepted and ready to land.
LG!
Repository:
rC Clang
https://reviews.llvm.org/D48532
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/c
rnkovacs created this revision.
rnkovacs added reviewers: NoQ, xazax.hun, george.karpenkov, dcoughlin.
Herald added subscribers: mikhail.ramalho, a.sidorin, dkrupp, szepet,
baloghadamsoftware, whisperity.
+ Cleaned up test file a bit.
Repository:
rC Clang
https://reviews.llvm.org/D48532
Fil
