Merge bitcoin/bitcoin#25830: refactor: Replace m_params with chainman.GetParams()

5d3f98d278 refactor: Replace m_params with chainman.GetParams() (Aurèle Oulès)

Pull request description:

  Fixes a TODO introduced in #24595.
  Removes `m_params` from `CChainState` class and replaces it with `m_chainman.GetParams()`.

ACKs for top commit:
  MarcoFalke:
    review ACK 5d3f98d278 🌎

Tree-SHA512: de0fe31450d281cc7307c0d820495e86c93c7998e77a148db2c703da66cff1059e6560c041f1864913c42075aa24d259c2623d45e929ca0a8056ed330a9f9978
This commit is contained in:
MacroFake
2022-10-19 10:04:27 +02:00
2 changed files with 38 additions and 35 deletions

View File

@@ -472,10 +472,6 @@ public:
//! Chainstate instances.
node::BlockManager& m_blockman;
/** Chain parameters for this chainstate */
/* TODO: replace with m_chainman.GetParams() */
const CChainParams& m_params;
//! The chainstate manager that owns this chainstate. The reference is
//! necessary so that this instance can check whether it is the active
//! chainstate within deeply nested method calls.