Charusso accepted this revision.
Charusso added inline comments.
================
Comment at: clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp:103
void enableWerror() { ShouldEmitAsError = true; }
+ void enableFixitsAsRemarks() { FixitsAsRemarks = true; }
----------------
NoQ wrote:
> Charusso wrote:
> > `FixIt` with capital `i`.
> It looks as if people write `FixIt` when they mean "fix-it" as an adjective
> (eg., "fix-it hint", "fix-it location") but `Fixit` when they mean "a fixit"
> as a noun.
The name of the stuff is `fix-it` as properly named in the title of the review
and `fixit` is a total different noun. I do not mind, if we pick `fixit`, but
the tooling will remain named as `fix-it`.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65182/new/
https://reviews.llvm.org/D65182
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
