ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: clang-tools-extra/clang-tidy/utils/TransformerClangTidyCheck.cpp:21
: ClangTidyCheck(Name, Context), Rule(std::move(R)) {
- for (const auto &Case : Rule.Cases) {
- assert(Case.Explanation != nullptr &&
- "clang-tidy checks must have an explanation by default;"
- " explicitly provide an empty explanation if none is desired");
- }
+ assert(std::all_of(Rule.Cases.begin(), Rule.Cases.end(),
+ [](const RewriteRule::Case &C) {
----------------
NIT: use `llvm::all_of` avoid typing the container name twice.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62412/new/
https://reviews.llvm.org/D62412
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
