Page MenuHomePhabricator

validation: LoadChainstate only needs Consensus::Params
ClosedPublic

Authored by Fabien on Nov 22 2022, 16:01.

Details

Reviewers
sdulfari
Group Reviewers
Restricted Project
Commits
rABC21ce55910141: 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

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable