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

Reply via email to