[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.
Repository:
rL LLVM
https://reviews.llvm.org/D38985
More information about the cfe-commits
mailing list