From b49dcbedf79613f0e0f61bfd742ed265213ed280 Mon Sep 17 00:00:00 2001 From: Jeremy Rubin Date: Fri, 6 Sep 2019 09:15:49 -0700 Subject: [PATCH] update variable naming conventions for IsTrusted --- src/wallet/wallet.cpp | 20 ++++++++++---------- src/wallet/wallet.h | 2 +- 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp index a14d538fe6d..22043810795 100644 --- a/src/wallet/wallet.cpp +++ b/src/wallet/wallet.cpp @@ -2299,7 +2299,7 @@ bool CWalletTx::IsTrusted(interfaces::Chain::Lock& locked_chain) const return IsTrusted(locked_chain, s); } -bool CWalletTx::IsTrusted(interfaces::Chain::Lock& locked_chain, std::set& trustedParents) const +bool CWalletTx::IsTrusted(interfaces::Chain::Lock& locked_chain, std::set& trusted_parents) const { // Quick answer in most cases if (!locked_chain.checkFinalTx(*tx)) { @@ -2329,12 +2329,12 @@ bool CWalletTx::IsTrusted(interfaces::Chain::Lock& locked_chain, std::setIsMine(parentOut) != ISMINE_SPENDABLE) return false; // If we've already trusted this parent, continue - if (trustedParents.count(parent->GetHash())) + if (trusted_parents.count(parent->GetHash())) continue; // Recurse to check that the parent is also trusted - if (!parent->IsTrusted(locked_chain, trustedParents)) + if (!parent->IsTrusted(locked_chain, trusted_parents)) return false; - trustedParents.insert(parent->GetHash()); + trusted_parents.insert(parent->GetHash()); } return true; } @@ -2420,11 +2420,11 @@ CWallet::Balance CWallet::GetBalance(const int min_depth, bool avoid_reuse) cons { auto locked_chain = chain().lock(); LOCK(cs_wallet); - std::set trustedParents; + std::set trusted_parents; for (const auto& entry : mapWallet) { const CWalletTx& wtx = entry.second; - const bool is_trusted{wtx.IsTrusted(*locked_chain, trustedParents)}; + const bool is_trusted{wtx.IsTrusted(*locked_chain, trusted_parents)}; const int tx_depth{wtx.GetDepthInMainChain(*locked_chain)}; const CAmount tx_credit_mine{wtx.GetAvailableCredit(*locked_chain, /* fUseCache */ true, ISMINE_SPENDABLE | reuse_filter)}; const CAmount tx_credit_watchonly{wtx.GetAvailableCredit(*locked_chain, /* fUseCache */ true, ISMINE_WATCH_ONLY | reuse_filter)}; @@ -2471,7 +2471,7 @@ void CWallet::AvailableCoins(interfaces::Chain::Lock& locked_chain, std::vector< const int min_depth = {coinControl ? coinControl->m_min_depth : DEFAULT_MIN_DEPTH}; const int max_depth = {coinControl ? coinControl->m_max_depth : DEFAULT_MAX_DEPTH}; - std::set trustedParents; + std::set trusted_parents; for (const auto& entry : mapWallet) { const uint256& wtxid = entry.first; @@ -2493,7 +2493,7 @@ void CWallet::AvailableCoins(interfaces::Chain::Lock& locked_chain, std::vector< if (nDepth == 0 && !wtx.InMempool()) continue; - bool safeTx = wtx.IsTrusted(locked_chain, trustedParents); + bool safeTx = wtx.IsTrusted(locked_chain, trusted_parents); // We should not consider coins from transactions that are replacing // other transactions. @@ -3778,12 +3778,12 @@ std::map CWallet::GetAddressBalances(interfaces::Chain: { LOCK(cs_wallet); - std::set trustedParents; + std::set trusted_parents; for (const auto& walletEntry : mapWallet) { const CWalletTx& wtx = walletEntry.second; - if (!wtx.IsTrusted(locked_chain, trustedParents)) + if (!wtx.IsTrusted(locked_chain, trusted_parents)) continue; if (wtx.IsImmatureCoinBase(locked_chain)) diff --git a/src/wallet/wallet.h b/src/wallet/wallet.h index 986baa89ee8..3513ebb91c6 100644 --- a/src/wallet/wallet.h +++ b/src/wallet/wallet.h @@ -616,7 +616,7 @@ public: bool InMempool() const; bool IsTrusted(interfaces::Chain::Lock& locked_chain) const; - bool IsTrusted(interfaces::Chain::Lock& locked_chain, std::set& trustedParents) const; + bool IsTrusted(interfaces::Chain::Lock& locked_chain, std::set& trusted_parents) const; int64_t GetTxTime() const;