From 6e0bc3fce45bddf96a88da669d27109775d09fe9 Mon Sep 17 00:00:00 2001 From: Alexander Shaposhnikov Date: Fri, 12 May 2017 00:16:56 +0000 Subject: [tooling] RefactoringCallbacks code cleanup This diff 1. adds missing "explicit" for single argument constructors 2. adds missing std::move in ReplaceNodeWithTemplate constructor Test plan: make check-all Differential revision: https://reviews.llvm.org/D33061 llvm-svn: 302855 --- clang/lib/Tooling/RefactoringCallbacks.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'clang/lib/Tooling/RefactoringCallbacks.cpp') diff --git a/clang/lib/Tooling/RefactoringCallbacks.cpp b/clang/lib/Tooling/RefactoringCallbacks.cpp index ad0a10dab9b..9fd333ca554 100644 --- a/clang/lib/Tooling/RefactoringCallbacks.cpp +++ b/clang/lib/Tooling/RefactoringCallbacks.cpp @@ -38,7 +38,7 @@ void ASTMatchRefactorer::addDynamicMatcher( class RefactoringASTConsumer : public ASTConsumer { public: - RefactoringASTConsumer(ASTMatchRefactorer &Refactoring) + explicit RefactoringASTConsumer(ASTMatchRefactorer &Refactoring) : Refactoring(Refactoring) {} void HandleTranslationUnit(ASTContext &Context) override { @@ -153,8 +153,8 @@ void ReplaceIfStmtWithItsBody::run( } ReplaceNodeWithTemplate::ReplaceNodeWithTemplate( - llvm::StringRef FromId, std::vector &&Template) - : FromId(FromId), Template(Template) {} + llvm::StringRef FromId, std::vector Template) + : FromId(FromId), Template(std::move(Template)) {} llvm::Expected> ReplaceNodeWithTemplate::create(StringRef FromId, StringRef ToTemplate) { -- cgit v1.2.3