aaron.ballman added inline comments.
================ Comment at: clang/docs/LibASTMatchersReference.html:91-94 +This mode is hard to use correctly and +requires more development iteration because it means +that the user must write AST Matchers to explicitly traverse or ignore nodes +which are not visible.</p> ---------------- I was trying to reword this to avoid making a judgmental statement that the default mode is hard to use correctly -- I'd strike this part of the paragraph. WDYT? ================ Comment at: clang/docs/LibASTMatchersReference.html:303 +<tr> + <td>Replacement of `begin()` with `cbegin()`: +<pre> ---------------- steveire wrote: > aaron.ballman wrote: > > Backticks won't help here -- should probably use <pre> tags. > `pre` is equivalent to three backticks. I added a `span` for use within > paragraphs. Thank you for translating my suggestion into the one I was trying to make. :-) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91639/new/ https://reviews.llvm.org/D91639 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
