================
@@ -30,9 +32,15 @@ void ContainerContainsCheck::registerMatchers(MatchFinder
*Finder) {
ofClass(cxxRecordDecl(HasContainsMatchingParamType)))))
.bind("call");
+ const auto Literal0 = integerLiteral(equals(0));
+ const auto Literal1 = integerLiteral(equals(1));
+
const auto FindCall =
cxxMemberCallExpr(
- argumentCountIs(1),
+ anyOf(argumentCountIs(1),
+ allOf(argumentCountIs(2), // string::find takes two arguments
----------------
nicovank wrote:
```suggestion
// `find` for string-like classes often has a second parameter,
// indicating where to start the search. We match if it's
// the default zero or explicitly equal to zero.
allOf(argumentCountIs(2),
```
I think a little more detail is good here.
https://github.com/llvm/llvm-project/pull/110351
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
