Szelethus marked an inline comment as done.
Szelethus added inline comments.
================
Comment at: clang/lib/StaticAnalyzer/Core/BugReporter.cpp:111
+/// getters and some well placed asserts for extra security.
+class BugReportConstruct {
+ /// The consumer we're constructing the bug report for.
----------------
NoQ wrote:
> You mean `Construct` like a noun? Not sure what that means.
>
> Maybe `PathDiagnosticContext`?
Yea, I originally wanted to have the class name `BaconStrip` so that it screams
just a bit louder that I couldnt come up with anything good, but maybe
`PathDiagnosticConstruction` or something like that would be better.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65484/new/
https://reviews.llvm.org/D65484
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
