Chainparams: Explicit Consensus::Params arg in consensus functions #6024

pull jtimon wants to merge 1 commits into bitcoin:master from jtimon:params_consensus2 changing 3 files +27 −22

jtimon commented at 2:03 PM on April 17, 2015: contributor

Explicit Consensus::Params arg for the main BlockHeader and Block validation functions in main.o:

-CheckBlock -CheckBlockHeader -ContextualCheckBlock -ContextualCheckBlockHeader

This is part of #5970. This functions have been chosen to be adapted first because they will be part of libconsensus. Note that although #5995 and #5946 don't depend on this, merging this would make some of their commits easier to review (the ones that adapt these functions for consensus, which are not part of non-dependent PRs yet).

Chainparams: Consensus: Explicit Consensus::Params arg for:

main:
-CheckBlock
-CheckBlockHeader
-ContextualCheckBlock
-ContextualCheckBlockHeader

23f9586f42

jtimon cross-referenced this on Apr 17, 2015 from issue DEPENDENT: Globals: Avoid calling Params() by jtimon
jtimon commented at 4:10 PM on April 22, 2015: contributor

Closing for now
jtimon closed this on Apr 22, 2015
jtimon cross-referenced this on Aug 20, 2015 from issue Move blocksize and related parameters to consensusparams by theuni
jtimon cross-referenced this on Aug 26, 2015 from issue Consensus: Adapt declarations of most obviously consensus functions by jtimon
jtimon cross-referenced this on Sep 14, 2015 from issue Consensus: Separate most consensus functions to consensus.cpp by jtimon
jtimon cross-referenced this on Apr 6, 2016 from issue Globals: TODO: Experiment: Kill "Params()" by jtimon
bitcoin locked this on Sep 8, 2021

Contributors