[PATCH] D63784: [clang-tidy] Fix ClangTidyTest to initialize context before checks.

Yitzhak Mandelbaum via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 26 07:43:40 PDT 2019


ymandel added inline comments.


================
Comment at: clang-tools-extra/unittests/clang-tidy/ClangTidyTest.h:48
+
+template <typename... CheckList>
 class TestClangTidyAction : public ASTFrontendAction {
----------------
gribozavr wrote:
> "CheckTypes"?  'cause "Checks" below is also technically a "check list".
Agreed. I was just blindly following what was already there. Changed here and below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63784





More information about the cfe-commits mailing list