[PATCH] D38985: [refactor] Add support for editor commands that connect IDEs/editors to the refactoring actions

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 27 09:49:26 PDT 2017

arphaman added inline comments.

Comment at: lib/Tooling/Refactoring/RefactoringActions.cpp:28
+// FIXME: Remove the Actions alltogether.
+class ExtractRefactoring final : public RefactoringAction {
ioeric wrote:
> Maybe I'm missing the context here... Why do we want to remove the actions?
There's now some redundancy between the command/description in action and the actual refactoring classes like ExtractFunction.
Remove might be a bad term here, something like rewrite would be better. I will change this in the commit.



More information about the cfe-commits mailing list