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
