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