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

Gábor Horváth via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 8 13:24:57 PDT 2016


xazax.hun added a comment.

In https://reviews.llvm.org/D24349#537594, @omtcyfz wrote:

> 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`.


I agree with Eugene. But I think a good consensus could be to warn on the non-const case but do not do the rewrite. What do you think?


Repository:
  rL LLVM

https://reviews.llvm.org/D24349





More information about the cfe-commits mailing list