Eugene.Zelenko added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/utils/TransformerTidy.cpp:22 + // Verify the existence and validity of the AST node that roots this rule. + auto &NodesMap = Result.Nodes.getMap(); + auto Root = NodesMap.find(tooling::RewriteRule::RootId); ---------------- Please don't use auto when return type is not spelled at same statement or iterator. Same for other places. ================ Comment at: clang-tools-extra/unittests/clang-tidy/TransformerTidyTest.cpp:10 +#include "../clang-tidy/utils/TransformerTidy.h" + +#include "ClangTidyTest.h" ---------------- Unnecessary empty line. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61386/new/ https://reviews.llvm.org/D61386 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits