Eugene.Zelenko added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/modernize/UseEmplaceCheck.cpp:25 + Names) { + const auto FullName = "::" + Node.getQualifiedNameAsString(); + ---------------- Please don't use auto unless type is spelled explicitly in same statement or iterator. ================ Comment at: clang-tools-extra/clang-tidy/modernize/UseEmplaceCheck.cpp:39 + + const auto FullNameTrimmedRef = StringRef(FullNameTrimmed); + ---------------- Please don't use auto unless type is spelled explicitly in same statement or iterator. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101471/new/ https://reviews.llvm.org/D101471 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits