JonasToth marked an inline comment as done.
JonasToth added a comment.
ping @aaron.ballman just in case the patch is now down on the list, I am sorry
for my high latency :(
================
Comment at:
clang-tools-extra/clang-tidy/cppcoreguidelines/ConstCorrectnessCheck.h:29
+ : ClangTidyCheck(Name, Context),
+ AnalyzeValues(Options.get("AnalyzeValues", 1)),
+ AnalyzeReferences(Options.get("AnalyzeReferences", 1)),
----------------
njames93 wrote:
> Can all these options be defaulted to Boolean values, as the options parser
> has special parsing logic for bool
you are right. I updated it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D54943/new/
https://reviews.llvm.org/D54943
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
