klimek added inline comments.
================
Comment at:
cfe/trunk/include/clang/Tooling/Refactoring/RefactoringActionRule.h:23-37
+/// A common refactoring action rule interface.
+class RefactoringActionRule {
+public:
+ enum RuleKind { SourceChangeRefactoringRuleKind };
+
+ RuleKind getRuleKind() const { return Kind; }
+
----------------
Sorry for being late, was out on vacation.
Generally, why do we need this tag-based abstraction here instead of using the
more typical OO double-dispatch where necessary?
(We do this in the AST a lot, but the AST is special, because there we want to
implement a lot of different algorithms that rely on the node type, while I
don't see how that applies here)
Repository:
rL LLVM
https://reviews.llvm.org/D36075
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
