Replace some more erase/remove patterns with removeIf

Change-Id: I6c6f33450fbfd798e5ef71a8ba18f590581dc3a2
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
This commit is contained in:
Giuseppe D'Angelo 2021-01-01 18:03:02 +01:00
parent b211148e4b
commit c34c6af374
4 changed files with 5 additions and 12 deletions

View File

@ -1501,9 +1501,7 @@ void TorrentClient::addToPeerList(const QList<TorrentPeer> &peerList)
};
// Remove inactive peers from the peer list until we're below
// the max connections count.
d->peers.erase(std::remove_if(d->peers.begin(), d->peers.end(),
firstNInactivePeers),
d->peers.end());
d->peers.removeIf(firstNInactivePeers);
// If we still have too many peers, remove the oldest ones.
d->peers.erase(d->peers.begin(), d->peers.begin() + tooMany);
}

View File

@ -1508,7 +1508,7 @@ QMakeEvaluator::VisitReturn QMakeEvaluator::evaluateBuiltinConditional(
auto isFrom = [pro](const ProString &s) {
return s.sourceFile() == pro;
};
vit->erase(std::remove_if(vit->begin(), vit->end(), isFrom), vit->end());
vit->removeIf(isFrom);
if (vit->isEmpty()) {
// When an initially non-empty variable becomes entirely empty,
// undefine it altogether.

View File

@ -641,10 +641,7 @@ QList<QPair<int, int>> QSortFilterProxyModelPrivate::proxy_intervals_for_source_
interval.first = interval.second = -1;
}
}
proxy_intervals.erase(
std::remove_if(proxy_intervals.begin(), proxy_intervals.end(),
[](QPair<int, int> &interval) { return interval.first < 0; }),
proxy_intervals.end());
proxy_intervals.removeIf([](QPair<int, int> interval) { return interval.first < 0; });
return proxy_intervals;
}

View File

@ -5204,10 +5204,8 @@ void tst_QWidget::setWindowGeometry_data()
const bool skipEmptyRects = (m_platform == QStringLiteral("windows"));
for (Rects l : qAsConst(rects)) {
if (skipEmptyRects) {
l.erase(std::remove_if(l.begin(), l.end(), [] (const QRect &r) { return r.isEmpty(); }),
l.end());
}
if (skipEmptyRects)
l.removeIf([] (const QRect &r) { return r.isEmpty(); });
const QRect &rect = l.constFirst();
for (int windowFlag : windowFlags) {
QTest::newRow(QString("%1,%2 %3x%4, flags %5")