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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to