LegalizeAdulthood added inline comments.
================ Comment at: clang-tools-extra/docs/clang-tidy/Contributing.rst:295 +Reference <https://clang.llvm.org/docs/LibASTMatchersReference.html>`_ to understand +the relationship between the different matcher funtions. + ---------------- Should be `functions` ================ Comment at: clang-tools-extra/docs/clang-tidy/Contributing.rst:338 +matching expressions to simplify your matcher. Just like breaking up a huge function +into smaller chunks with an intention revealing name can help you understand a complex +algorithm, breaking up a matcher into smaller matchers with intention revealing names ---------------- chunks plural, "an intention revealing name" singular mismatch CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117939/new/ https://reviews.llvm.org/D117939 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
