alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land.
LG with a nit. ================ 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. + ---------------- 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"? 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