hokein added inline comments.
================ Comment at: include/clang/Tooling/Refactoring/Rename/RenamingAction.h:79 QualifiedRenameRule(const NamedDecl *ND, - std::string NewQualifiedName) + const std::string &NewQualifiedName) : ND(ND), NewQualifiedName(std::move(NewQualifiedName)) {} ---------------- Passing `std::string` object is fine here -- because we use std::move below to avoid the extra copy. Is the warning caught by the clang-tidy check? https://reviews.llvm.org/D40543 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits