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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
