Charusso added a comment.
Thanks for the reviews! Are we good to go?
================
Comment at: clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp:112
void enableFixitsAsRemarks() { FixitsAsRemarks = true; }
+ void enableFixItApplication() { ApplyFixIts = true; }
----------------
alexfh wrote:
> nit: I'd suggest naming the method closer to the name of the corresponding
> field, e.g. `enableApplyFixIts`. Why isn't this `setApplyFixIts(bool)` btw?
We do not support the disable way of options, so let us make it
`enableApplyFixIts()`.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69746/new/
https://reviews.llvm.org/D69746
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
