[PATCH] D112646: [clang-tidy] Add `readability-container-contains` check

[Whisperity via Phabricator via cfe-commits]

whisperity added a comment.

In D112646#3104236 <https://reviews.llvm.org/D112646#3104236>, @avogelsgesang wrote:

> So I guess the name would have to be `container-count-begin-end-contains` or similar... which would be a bit much in my opinion

Yeah, that would be too much indeed.

However, consider `readability-container-use-contains`. It indicates a bit more that we want people to write terse code that is clear about its intentions.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112646/new/

https://reviews.llvm.org/D112646