summaryrefslogtreecommitdiff
path: root/apt-pkg/solver3.cc
diff options
context:
space:
mode:
authorJulian Andres Klode <jak@debian.org>2025-12-29 21:26:24 +0100
committerJulian Andres Klode <jak@debian.org>2026-01-05 21:20:24 +0000
commit62fa3858d12f163de7db81f6e2be770e39783238 (patch)
treea628a7cc467f3d7a276fdda245c0e299752ae5d3 /apt-pkg/solver3.cc
parentf02f90021c255ef3aa578e00758cbc62e7fc82dd (diff)
solver3: Modernize std::find() to std::ranges::contains
Where applicable
Diffstat (limited to 'apt-pkg/solver3.cc')
-rw-r--r--apt-pkg/solver3.cc15
1 files changed, 6 insertions, 9 deletions
diff --git a/apt-pkg/solver3.cc b/apt-pkg/solver3.cc
index 9313b433d..096cbb521 100644
--- a/apt-pkg/solver3.cc
+++ b/apt-pkg/solver3.cc
@@ -826,17 +826,15 @@ const Clause *DependencySolver::RegisterClause(Clause &&clause)
earlierDep.TargetPkg() != dep.TargetPkg())
continue;
if (std::none_of(earlierClause->solutions.begin(), earlierClause->solutions.end(), [&clause](auto earlierSol)
- { return std::find(clause.solutions.begin(),
- clause.solutions.end(),
- earlierSol) != clause.solutions.end(); }))
+ { return std::ranges::contains(clause.solutions,
+ earlierSol); }))
continue;
if (earlierClause->optional == clause.optional)
{
std::erase_if(earlierClause->solutions, [&clause, this](auto earlierSol)
- { return std::find(clause.solutions.begin(),
- clause.solutions.end(),
- earlierSol) == clause.solutions.end(); });
+ { return not std::ranges::contains(clause.solutions,
+ earlierSol); });
earlierClause->merged.push_front(clause);
merged = true;
@@ -845,9 +843,8 @@ const Clause *DependencySolver::RegisterClause(Clause &&clause)
{
// If say a Depends has fewer solution than a Recommends, remove the Recommend's extranous ones.
std::erase_if(clause.solutions, [&earlierClause, this](auto sol)
- { return std::find(earlierClause->solutions.begin(),
- earlierClause->solutions.end(),
- sol) == earlierClause->solutions.end(); });
+ { return not std::ranges::contains(earlierClause->solutions,
+ sol); });
// Remove recursion here, such that we display correctly (if we ever display anywhere...)
auto earlierClauseCopy = *earlierClause;