[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)

Nicolas van Kempen via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 28 13:36:23 PDT 2024


================
@@ -94,12 +102,14 @@ void ContainerContainsCheck::registerMatchers(MatchFinder *Finder) {
       binaryOperator(hasLHS(Literal1), hasOperatorName(">"), hasRHS(CountCall))
           .bind("negativeComparison"));
 
-  // Find membership tests based on `find() == end()`.
+  // Find membership tests based on `find() == end() or find() == npos`.
----------------
nicovank wrote:

```suggestion
  // Find membership tests based on `find() == end()` or `find() == npos`.
```

https://github.com/llvm/llvm-project/pull/110351


More information about the cfe-commits mailing list