diff options
| author | Haojian Wu <hokein@google.com> | 2017-10-19 08:20:55 +0000 |
|---|---|---|
| committer | Haojian Wu <hokein@google.com> | 2017-10-19 08:20:55 +0000 |
| commit | 4fa23a100e74a202fc0140cc81e8c4090c806848 (patch) | |
| tree | 7e4309635fa8ae56c0c8b717f7a11a1aa6bda4a5 /clang/lib/Tooling/Refactoring/Rename | |
| parent | f309f0fd70df33687ff4ad19e4a7ef72c76c1441 (diff) | |
| download | bcm5719-llvm-4fa23a100e74a202fc0140cc81e8c4090c806848.tar.gz bcm5719-llvm-4fa23a100e74a202fc0140cc81e8c4090c806848.zip | |
Fix a few nits in RenamingAction.
* Add missing override keyword.
* avoid unnecessary copy of std::string.
llvm-svn: 316152
Diffstat (limited to 'clang/lib/Tooling/Refactoring/Rename')
| -rw-r--r-- | clang/lib/Tooling/Refactoring/Rename/RenamingAction.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/lib/Tooling/Refactoring/Rename/RenamingAction.cpp b/clang/lib/Tooling/Refactoring/Rename/RenamingAction.cpp index fdff6fd636a..28912c3e139 100644 --- a/clang/lib/Tooling/Refactoring/Rename/RenamingAction.cpp +++ b/clang/lib/Tooling/Refactoring/Rename/RenamingAction.cpp @@ -77,10 +77,10 @@ private: class RenameOccurrences final : public SourceChangeRefactoringRule { public: RenameOccurrences(const NamedDecl *ND, std::string NewName) - : Finder(ND), NewName(NewName) {} + : Finder(ND), NewName(std::move(NewName)) {} Expected<AtomicChanges> - createSourceReplacements(RefactoringRuleContext &Context) { + createSourceReplacements(RefactoringRuleContext &Context) override { Expected<SymbolOccurrences> Occurrences = Finder.findSymbolOccurrences(Context); if (!Occurrences) |
