arphaman marked 5 inline comments as done. arphaman added inline comments.
================ Comment at: include/clang/Tooling/Refactoring/RefactoringActionRules.h:33 +/// +/// - requiredSelection: The refactoring function won't be invoked unless the +/// given selection requirement is satisfied. ---------------- ioeric wrote: > We might want to document supported requirements somewhere else so that we > don't need to update this file every time a new requirement is added. Do you think it should be in Clang's documentation? I can start on a new document there but I'd prefer to do it in a separate patch. WDYT? 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