[clang-tools-extra] [clang-tidy] fix false positive in modernize-min-max-use-initializer-list (PR #107649)
Danny Mösch via cfe-commits
cfe-commits at lists.llvm.org
Sat Sep 7 12:35:07 PDT 2024
================
@@ -72,11 +72,10 @@ static FindArgsResult findArgs(const CallExpr *Call) {
return Result;
}
-static SmallVector<FixItHint>
-generateReplacements(const MatchFinder::MatchResult &Match,
- const CallExpr *TopCall, const FindArgsResult &Result,
- const bool IgnoreNonTrivialTypes,
- const std::uint64_t IgnoreTrivialTypesOfSizeAbove) {
+static std::pair<bool, SmallVector<FixItHint>> generateReplacements(
+ const MatchFinder::MatchResult &Match, const CallExpr *TopCall,
+ const FindArgsResult &Result, const bool IgnoreNonTrivialTypes,
+ const std::uint64_t IgnoreTrivialTypesOfSizeAbove, bool FoundNestedCall) {
----------------
SimplyDanny wrote:
All callers use `false` as the last argument. Is the new argument needed at all?
https://github.com/llvm/llvm-project/pull/107649
More information about the cfe-commits
mailing list