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

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 14 01:11:50 PDT 2016


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





More information about the cfe-commits mailing list