Skip to content

Commit 2c0c3f8

Browse files
author
MarcoFalke
committed
Merge bitcoin#19217: p2p: disambiguate block-relay-only variable names from blocksonly variables
ec4c6a1 scripted-diff: replace MAX_BLOCKS_ONLY_CONNECTIONS with MAX_BLOCK_RELAY_ONLY_CONNECTIONS (glowang) Pull request description: We have two different concepts that have similar names: `-blocksonly` and `block-relay-only`, and the similarity of names could lead to confusion. `-blocksonly` disables all local receiving & relaying of transactions (with a few exceptions), while `block-relay-only`means that bitcoind will make 2 additional outbound connections that are only used for block relay. In net.h and init.cpp, `MAX_BLOCKS_ONLY_CONNECTIONS` is used to represent the maximum number of `block-relay-only` outbound peers, which is 2. But this name sounds ambiguous, and I proposed a better name, `MAX_BLOCK_RELAY_ONLY_CONNECTION`. ACKs for top commit: jnewbery: ACK ec4c6a1 Tree-SHA512: cfa592a7ff936f14d10cfc1e926a51b82bc0feaf104885a41ca8111b906cb3d1ec5536bab143a3cfca70aa49e9575c6995941eb6d3d7f4018d4535712342f155
2 parents edfeaf6 + ec4c6a1 commit 2c0c3f8

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/init.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1888,7 +1888,7 @@ bool AppInitMain(const util::Ref& context, NodeContext& node)
18881888
connOptions.nLocalServices = nLocalServices;
18891889
connOptions.nMaxConnections = nMaxConnections;
18901890
connOptions.m_max_outbound_full_relay = std::min(MAX_OUTBOUND_FULL_RELAY_CONNECTIONS, connOptions.nMaxConnections);
1891-
connOptions.m_max_outbound_block_relay = std::min(MAX_BLOCKS_ONLY_CONNECTIONS, connOptions.nMaxConnections-connOptions.m_max_outbound_full_relay);
1891+
connOptions.m_max_outbound_block_relay = std::min(MAX_BLOCK_RELAY_ONLY_CONNECTIONS, connOptions.nMaxConnections-connOptions.m_max_outbound_full_relay);
18921892
connOptions.nMaxAddnode = MAX_ADDNODE_CONNECTIONS;
18931893
connOptions.nMaxFeeler = MAX_FEELER_CONNECTIONS;
18941894
connOptions.nBestHeight = chain_active_height;

src/net.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ static const int MAX_OUTBOUND_FULL_RELAY_CONNECTIONS = 8;
6161
/** Maximum number of addnode outgoing nodes */
6262
static const int MAX_ADDNODE_CONNECTIONS = 8;
6363
/** Maximum number of block-relay-only outgoing connections */
64-
static const int MAX_BLOCKS_ONLY_CONNECTIONS = 2;
64+
static const int MAX_BLOCK_RELAY_ONLY_CONNECTIONS = 2;
6565
/** Maximum number of feeler connections */
6666
static const int MAX_FEELER_CONNECTIONS = 1;
6767
/** -listen default */

0 commit comments

Comments
 (0)