[PATCH] D69746: [analyzer] FixItHint: Apply and test hints with the Clang-Tidy's script

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 25 03:02:51 PST 2020


NoQ added inline comments.


================
Comment at: clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp:92
+
+  bool IncludePath = false, ShouldEmitAsError = false, FixitsAsRemarks = false,
+       ApplyFixIts = false;
----------------
Charusso wrote:
> NoQ wrote:
> > I'll be perfectly happy with removing `FixitsAsRemarks` entirely. Your new mechanism is superior.
> Okai, challenge accepted. Thanks!
So, can we remove `FixitsAsRemarks` now or is anything still blocking it?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69746/new/

https://reviews.llvm.org/D69746





More information about the cfe-commits mailing list