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

Thu Oct 28 22:00:59 PDT 2021

xazax.hun added a comment.

In D112646#3092619 <https://reviews.llvm.org/D112646#3092619>, @whisperity wrote:

> @xazax.hun I think you did something similar wrt. `empty()`, right? Could you take a look at this?

The idea looks good to me. The module also sounds right. If we want to follow container-size-empty's convention, we should include the replaced method in the name. But I am not insisting on anything.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112646