ccotter added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp:158-159 + callee(cxxMethodDecl(BeginNameMatcher))), + callExpr(argumentCountIs(1), + callee(functionDecl(BeginNameMatcher))))) .bind(BeginCallName); ---------------- PiotrZSL wrote: > Now it will also catch calls to single parameter methods... > maybe `unless(cxxMethodDecl())`, after all we want to catch only functions I think `usesADL()` accomplishes the same thing here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140760/new/ https://reviews.llvm.org/D140760 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits