diff options
| author | Alexander Shaposhnikov <shal1t712@gmail.com> | 2017-05-12 00:16:56 +0000 |
|---|---|---|
| committer | Alexander Shaposhnikov <shal1t712@gmail.com> | 2017-05-12 00:16:56 +0000 |
| commit | 58af9a7cb612e50a213f00675c1604e644cb7421 (patch) | |
| tree | fd2d4b2ba8489ddaea82c3a5a1a2f4de93557526 /lib/Tooling/RefactoringCallbacks.cpp | |
| parent | 031053b0451a029db35118afda082d8b2cf8db5e (diff) | |
| download | clang-58af9a7cb612e50a213f00675c1604e644cb7421.tar.gz | |
[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
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@302855 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Tooling/RefactoringCallbacks.cpp')
| -rw-r--r-- | lib/Tooling/RefactoringCallbacks.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/Tooling/RefactoringCallbacks.cpp b/lib/Tooling/RefactoringCallbacks.cpp index ad0a10dab9..9fd333ca55 100644 --- a/lib/Tooling/RefactoringCallbacks.cpp +++ b/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<TemplateElement> &&Template) - : FromId(FromId), Template(Template) {} + llvm::StringRef FromId, std::vector<TemplateElement> Template) + : FromId(FromId), Template(std::move(Template)) {} llvm::Expected<std::unique_ptr<ReplaceNodeWithTemplate>> ReplaceNodeWithTemplate::create(StringRef FromId, StringRef ToTemplate) { |