diff options
author | glozow <gloriajzhao@gmail.com> | 2024-02-09 22:06:26 +0000 |
---|---|---|
committer | glozow <gloriajzhao@gmail.com> | 2024-02-12 14:27:25 +0000 |
commit | 63b62e123e38cb92c2135e63eb1a5b760c11dd4e (patch) | |
tree | 02b878e043997d9f932506de1c9963ca283c1ffb /src | |
parent | e3c17112ddfe740e5ab4c652d80ed1a19c13da5a (diff) | |
download | bitcoin-63b62e123e38cb92c2135e63eb1a5b760c11dd4e.tar.xz |
[doc] fix docs and comments from v3
Diffstat (limited to 'src')
-rw-r--r-- | src/policy/v3_policy.cpp | 5 | ||||
-rw-r--r-- | src/test/fuzz/tx_pool.cpp | 2 |
2 files changed, 2 insertions, 5 deletions
diff --git a/src/policy/v3_policy.cpp b/src/policy/v3_policy.cpp index 158881aeb9..f838dc6c0f 100644 --- a/src/policy/v3_policy.cpp +++ b/src/policy/v3_policy.cpp @@ -81,9 +81,9 @@ std::optional<std::string> PackageV3Checks(const CTransactionRef& ptx, int64_t v vsize, V3_CHILD_MAX_VSIZE); } + // Exactly 1 parent exists, either in mempool or package. Find it. const auto parent_info = [&] { if (mempool_ancestors.size() > 0) { - // There's a parent in the mempool. auto& mempool_parent = *mempool_ancestors.begin(); Assume(mempool_parent->GetCountWithDescendants() == 1); return ParentInfo{mempool_parent->GetTx().GetHash(), @@ -91,7 +91,6 @@ std::optional<std::string> PackageV3Checks(const CTransactionRef& ptx, int64_t v mempool_parent->GetTx().nVersion, /*has_mempool_descendant=*/mempool_parent->GetCountWithDescendants() > 1}; } else { - // Ancestor must be in the package. Find it. auto& parent_index = in_package_parents.front(); auto& package_parent = package.at(parent_index); return ParentInfo{package_parent->GetHash(), @@ -184,7 +183,7 @@ std::optional<std::string> SingleV3Checks(const CTransactionRef& ptx, // The rest of the rules only apply to transactions with nVersion=3. if (ptx->nVersion != 3) return std::nullopt; - // Check that V3_ANCESTOR_LIMIT would not be violated, including both in-package and in-mempool. + // Check that V3_ANCESTOR_LIMIT would not be violated. if (mempool_ancestors.size() + 1 > V3_ANCESTOR_LIMIT) { return strprintf("tx %s (wtxid=%s) would have too many ancestors", ptx->GetHash().ToString(), ptx->GetWitnessHash().ToString()); diff --git a/src/test/fuzz/tx_pool.cpp b/src/test/fuzz/tx_pool.cpp index b44b528b6f..fcf230642a 100644 --- a/src/test/fuzz/tx_pool.cpp +++ b/src/test/fuzz/tx_pool.cpp @@ -407,8 +407,6 @@ FUZZ_TARGET(tx_pool, .init = initialize_tx_pool) const bool accepted = res.m_result_type == MempoolAcceptResult::ResultType::VALID; if (accepted) { txids.push_back(tx->GetHash()); - // Only check fees if accepted and not bypass_limits, otherwise it's not guaranteed that - // trimming has happened for this tx and previous iterations. CheckMempoolV3Invariants(tx_pool); } } |