Merge bitcoin/bitcoin#29308: doc: update BroadcastTransaction comment

31cce4a1bdbb48f57996615ee6c686e456cc0bea doc: update `BroadcastTransaction` comment (ismaelsadeeq)

Pull request description:

  `BroadcastTransaction` is also called by `submitpackage` RPC.

  All transactions that are accepted into the mempool post package processing are broadcasted to peers individually here
  ea4ddd8652/src/rpc/mempool.cpp (L926)

  It's not maintainable to list all the callers of a function.

ACKs for top commit:
  stickies-v:
    ACK 31cce4a1bdbb48f57996615ee6c686e456cc0bea
  kristapsk:
    ACK 31cce4a1bdbb48f57996615ee6c686e456cc0bea
  naumenkogs:
    ACK 31cce4a1bdbb48f57996615ee6c686e456cc0bea

Tree-SHA512: 8aea92c53c1911a0ac36fe9e3a24d37d83e7d9b40a16f0832bfa7a719328697621e3f94a5dc80d1840e7ae705e0c3aab7a3df7064986e1e53a4a4114adf078a8
This commit is contained in:
glozow 2024-01-30 12:09:26 +00:00 committed by pasta
parent 2a77808596
commit 8bf1d06599
No known key found for this signature in database
GPG Key ID: E2F3D7916E722D38

View File

@ -31,7 +31,7 @@ static TransactionError HandleATMPError(const TxValidationState& state, std::str
TransactionError BroadcastTransaction(NodeContext& node, const CTransactionRef tx, bilingual_str& err_string, const CAmount& max_tx_fee, bool relay, bool wait_callback, bool bypass_limits) TransactionError BroadcastTransaction(NodeContext& node, const CTransactionRef tx, bilingual_str& err_string, const CAmount& max_tx_fee, bool relay, bool wait_callback, bool bypass_limits)
{ {
// BroadcastTransaction can be called by either sendrawtransaction RPC or the wallet. // BroadcastTransaction can be called by RPC or by the wallet.
// chainman, mempool and peerman are initialized before the RPC server and wallet are started // chainman, mempool and peerman are initialized before the RPC server and wallet are started
// and reset after the RPC sever and wallet are stopped. // and reset after the RPC sever and wallet are stopped.
assert(node.chainman); assert(node.chainman);