[PATCH] D65578: [analyzer][NFC] Make sure that the BugReport is not modified during the construction of non-visitor pieces
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 7 15:13:40 PDT 2019
NoQ accepted this revision.
NoQ added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h:108
SmallVector<SourceRange, 4> Ranges;
+ const SourceRange ErrorNodeRange;
ExtraTextList ExtraText;
----------------
I guess you should add a comment for anybody who would want to remove this `const` on why it's a bad idea(?)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65578/new/
https://reviews.llvm.org/D65578
More information about the cfe-commits
mailing list