HomePhabricator

refactor: use chainman instead of chainParams for DeploymentActive*

Description

refactor: use chainman instead of chainParams for DeploymentActive*

Summary:
This is a partial backport of core#24595
https://github.com/bitcoin/bitcoin/pull/24595/commits/deffe0df6c36225bada18603b5a840139f030f2c
https://github.com/bitcoin/bitcoin/pull/24595/commits/78adef17536edef833a0bfca06b61ce28120e486

We don't have a g_versionbitscache global, and the point of the PR in the source material is to make that global a ChainstateManager member. The only rationale for backporting this in Bitcoin ABC is to make following backports easier (for instance in core#25704 one of the functions touched here will use the chainman argument)

Depends on D15808

Test Plan: ninja all check-all

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D15809

Details

Provenance
PiRKAuthored on Mar 28 2024, 15:39
PiRKPushed on Mar 30 2024, 17:23
Reviewer
Restricted Project
Differential Revision
D15809: refactor: use chainman instead of chainParams for DeploymentActive*
Parents
rABC424d0fc3f6c1: get config via chainstatemanager when possible (5/5)
Branches
Unknown
Tags
Unknown