ioeric accepted this revision. ioeric added a comment. lg
================ Comment at: include/clang/Tooling/Refactoring/RefactoringActionRule.h:48 + + // static const RefactoringDescriptor &describe() = 0; }; ---------------- A comment would be nice here. ================ Comment at: lib/Tooling/Refactoring/RefactoringActions.cpp:28 + +// FIXME: Remove the Actions alltogether. +class ExtractRefactoring final : public RefactoringAction { ---------------- Maybe I'm missing the context here... Why do we want to remove the actions? 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