hokein added inline comments.
================ Comment at: clang-tidy/ClangTidy.cpp:297 + const auto &RegisteredCheckers = + AnalyzerOptions::getRegisteredCheckers(IncludeExperimentalCheckers); + bool AnalyzerChecksEnabled = false; ---------------- alexfh wrote: > hokein wrote: > > Since the default argument of this function is `false` already. Can we use > > `AnalyzerOptions::getRegisteredCheckers();` directly? > I wanted to be explicit about this choice on the caller side as well. I see, but `IncludeExperimentalCheckers` is not needed here. How about `AnalyzerOptions::getRegisteredCheckers(/*IncludeExperimental=*/false);`? https://reviews.llvm.org/D26311 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
