Eugene.Zelenko added inline comments. ================ Comment at: docs/ReleaseNotes.rst:94 @@ +93,3 @@ + <http://clang.llvm.org/extra/clang-tidy/checks/readability-container-size-empty.html>`_ check + supports arbitrary containers with ``empty()`` and ``size()`` methods. + ---------------- alexfh wrote: > The wording is a bit too broad. We should somehow hint that the methods are > not just arbitrary. Maybe "with suitable ``size()`` and ``empty()`` methods"? Will fix on commit. **Suitable** sounds good enough since link to documentation is provided, where more details could be found.
Repository: rL LLVM https://reviews.llvm.org/D24526 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
