diff --git a/src/addrman.cpp b/src/addrman.cpp index f15293757ae..b879a73b515 100644 --- a/src/addrman.cpp +++ b/src/addrman.cpp @@ -516,7 +516,7 @@ void AddrManImpl::MakeTried(AddrInfo& info, nid_type nId) if (vvTried[nKBucket][nKBucketPos] != -1) { // find an item to evict nid_type nIdEvict = vvTried[nKBucket][nKBucketPos]; - assert(mapInfo.count(nIdEvict) == 1); + assert(mapInfo.contains(nIdEvict)); AddrInfo& infoOld = mapInfo[nIdEvict]; // Remove the to-be-evicted item from the tried set. @@ -919,7 +919,7 @@ void AddrManImpl::ResolveCollisions_() bool erase_collision = false; // If id_new not found in mapInfo remove it from m_tried_collisions - if (mapInfo.count(id_new) != 1) { + if (!mapInfo.contains(id_new)) { erase_collision = true; } else { AddrInfo& info_new = mapInfo[id_new]; @@ -985,7 +985,7 @@ std::pair AddrManImpl::SelectTriedCollision_() nid_type id_new = *it; // If id_new not found in mapInfo remove it from m_tried_collisions - if (mapInfo.count(id_new) != 1) { + if (!mapInfo.contains(id_new)) { m_tried_collisions.erase(it); return {}; } diff --git a/src/init.cpp b/src/init.cpp index bfb9483ad87..02b0826dac2 100644 --- a/src/init.cpp +++ b/src/init.cpp @@ -968,7 +968,7 @@ bool AppInitParameterInteraction(const ArgsManager& args) // Signal NODE_COMPACT_FILTERS if peerblockfilters and basic filters index are both enabled. if (args.GetBoolArg("-peerblockfilters", DEFAULT_PEERBLOCKFILTERS)) { - if (g_enabled_filter_types.count(BlockFilterType::BASIC) != 1) { + if (!g_enabled_filter_types.contains(BlockFilterType::BASIC)) { return InitError(_("Cannot set -peerblockfilters without -blockfilterindex.")); } diff --git a/src/qt/walletframe.cpp b/src/qt/walletframe.cpp index 7a8a053900b..269c9b4e4a6 100644 --- a/src/qt/walletframe.cpp +++ b/src/qt/walletframe.cpp @@ -70,7 +70,7 @@ bool WalletFrame::addView(WalletView* walletView) { if (!clientModel) return false; - if (mapWalletViews.count(walletView->getWalletModel()) > 0) return false; + if (mapWalletViews.contains(walletView->getWalletModel())) return false; walletView->setClientModel(clientModel); walletView->showOutOfSyncWarning(bOutOfSync); @@ -90,7 +90,7 @@ bool WalletFrame::addView(WalletView* walletView) void WalletFrame::setCurrentWallet(WalletModel* wallet_model) { - if (mapWalletViews.count(wallet_model) == 0) return; + if (!mapWalletViews.contains(wallet_model)) return; // Stop the effect of hidden widgets on the size hint of the shown one in QStackedWidget. WalletView* view_about_to_hide = currentWalletView(); @@ -116,7 +116,7 @@ void WalletFrame::setCurrentWallet(WalletModel* wallet_model) void WalletFrame::removeWallet(WalletModel* wallet_model) { - if (mapWalletViews.count(wallet_model) == 0) return; + if (!mapWalletViews.contains(wallet_model)) return; WalletView *walletView = mapWalletViews.take(wallet_model); walletStack->removeWidget(walletView); diff --git a/src/txrequest.cpp b/src/txrequest.cpp index 4d7240bee0d..bb013b0fa7b 100644 --- a/src/txrequest.cpp +++ b/src/txrequest.cpp @@ -580,7 +580,7 @@ public: // Bail out if we already have a CANDIDATE_BEST announcement for this (txhash, peer) combination. The case // where there is a non-CANDIDATE_BEST announcement already will be caught by the uniqueness property of the // ByPeer index when we try to emplace the new object below. - if (m_index.get().count(ByPeerView{peer, true, gtxid.ToUint256()})) return; + if (m_index.get().contains(ByPeerView{peer, true, gtxid.ToUint256()})) return; // Try creating the announcement with CANDIDATE_DELAYED state (which will fail due to the uniqueness // of the ByPeer index if a non-CANDIDATE_BEST announcement already exists with the same txhash and peer). diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp index 48601a59fff..5e285b23b5e 100644 --- a/src/wallet/wallet.cpp +++ b/src/wallet/wallet.cpp @@ -265,7 +265,7 @@ void WaitForDeleteWallet(std::shared_ptr&& wallet) wallet.reset(); { WAIT_LOCK(g_wallet_release_mutex, lock); - while (g_unloading_wallet_set.count(name) == 1) { + while (g_unloading_wallet_set.contains(name)) { g_wallet_release_cv.wait(lock); } } @@ -1525,7 +1525,7 @@ void CWallet::blockDisconnected(const interfaces::BlockInfo& block) for (const CTxIn& tx_in : ptx->vin) { // No other wallet transactions conflicted with this transaction - if (mapTxSpends.count(tx_in.prevout) < 1) continue; + if (!mapTxSpends.contains(tx_in.prevout)) continue; std::pair range = mapTxSpends.equal_range(tx_in.prevout);