arphaman added inline comments.
================ Comment at: include/clang/Tooling/Refactoring/RefactoringActionRuleRequirements.h:73 +template <typename OptionType> +class OptionRequirement : public RefactoringOptionsRequirement { +public: ---------------- ioeric wrote: > nit: `OptionRequirement` sounds more general than the base class > `RefactoringOptionsRequirement`. I couldn't really think of a better name, sorry. Repository: rL LLVM https://reviews.llvm.org/D37856 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits