[PATCH] D24526: [Release notes] Mention readability-container-size-empty improvements

Eugene Zelenko via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 14 10:46:50 PDT 2016


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





More information about the cfe-commits mailing list