HomePhabricator

validation: LoadChainstate only needs Consensus::Params

Description

validation: LoadChainstate only needs Consensus::Params

Summary:
We used to pass Config before the verification was split.

The original commit is about changing the signature of VerifyDB, but this is not applicable to our codebase because we need the config to get the max block size.

Partial backport of core#23280:
https://github.com/bitcoin/bitcoin/pull/23280/commits/15f2e33bb3d1ad3bc997f6a84956337f46495091

Depends on D12578.

Test Plan:

ninja all check-all

Reviewers: #bitcoin_abc, sdulfari

Reviewed By: #bitcoin_abc, sdulfari

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

Details

Provenance
Carl Dong <contact@carldong.me>Authored on Sep 20 2021, 18:02
FabienCommitted on Nov 22 2022, 19:38
FabienPushed on Nov 22 2022, 19:38
Reviewer
Restricted Project
Differential Revision
D12579: validation: LoadChainstate only needs Consensus::Params
Parents
rABC8d9a8aa4ce6c: node/chainstate: Decouple from ShutdownRequested
Branches
Unknown
Tags
Unknown