Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge #28148: refactor: consistently use ApplyArgsManOptions for Peer…
…Manager::Options 8a31597 refactor: deduplicate ignores_incoming_txs (stickies-v) 5f41afc refactor: set ignore_incoming_txs in ApplyArgsManOptions (stickies-v) Pull request description: Consistently use `ApplyArgsManOptions` for `PeerManager::Options`, and initialize `PeerManager::Options` early to avoid reading `"-blocksonly"` twice. Suggested in #27499 (comment) and also requested in #27499 (comment). No behaviour change, but the [`TestingSetup`](https://github.com/bitcoin/bitcoin/blob/e35fb7bc48d360585b80d0c7f89ac5087c1d405e/src/test/util/setup_common.cpp#L255-L256) is now also able to access `"-blocksonly"`. ACKs for top commit: MarcoFalke: lgtm ACK 8a31597 achow101: ACK 8a31597 TheCharlatan: ACK 8a31597 dergoegge: utACK 8a31597 Tree-SHA512: 6cb489d79ac2a87e8faedb76c96973ab3fc597426f274a90a3ffd0bc5fe3f2b25db9c7ec2e55a0c806c2bcbc0fdded6e228adb43d2cd81f14fd6552863847698
- Loading branch information