Re: r316152 - Fix a few nits in RenamingAction.

2017-10-19 Thread Alex L via cfe-commits
Thanks! On 19 October 2017 at 01:20, Haojian Wu via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: hokein > Date: Thu Oct 19 01:20:55 2017 > New Revision: 316152 > > URL: http://llvm.org/viewvc/llvm-project?rev=316152&view=rev > Log: > Fix a few nits in RenamingAction. > > * Add missi

r316152 - Fix a few nits in RenamingAction.

2017-10-19 Thread Haojian Wu via cfe-commits
Author: hokein Date: Thu Oct 19 01:20:55 2017 New Revision: 316152 URL: http://llvm.org/viewvc/llvm-project?rev=316152&view=rev Log: Fix a few nits in RenamingAction. * Add missing override keyword. * avoid unnecessary copy of std::string. Modified: cfe/trunk/lib/Tooling/Refactoring/Rename/R