Eugene.Zelenko added inline comments.
================ Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:29 + +namespace { +std::string GetIsolatedDecl(const VarDecl *D, const SourceManager &SM, ---------------- Please use static instead. ================ Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:37 + VarType.getAsString(TypePrinter) + " " + D->getNameAsString(); + std::string Initializer = ""; + ---------------- See readability-redundant-string-init. ================ Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:60 + const auto *WholeDecl = Result.Nodes.getNodeAs<DeclStmt>("decl_stmt"); + std::string Replacement = ""; + ---------------- See readability-redundant-string-init. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D51949 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits