[PATCH] D24349: [clang-tidy] Extend readability-container-size-empty to arbitrary class with size() and empty()

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 8 13:22:55 PDT 2016


omtcyfz added a comment.

In https://reviews.llvm.org/D24349#537589, @Eugene.Zelenko wrote:

> If size() and empty() change object's state, it may be not equivalent replacement.


True. But my point is that they are not required to do that if they're just not marked `const`.


Repository:
  rL LLVM

https://reviews.llvm.org/D24349





More information about the cfe-commits mailing list