Globals: Pass Consensus::Params through CBlockTreeDB::LoadBlockIndexGuts() #9176
pull jtimon wants to merge 1 commits into bitcoin:master from jtimon:0.13-chainparams-loadblockindexguts changing 3 files +6 −6-
jtimon commented at 6:45 am on November 17, 2016: contributorPart of #7829
-
jtimon force-pushed on Nov 17, 2016
-
in src/txdb.cpp: in 2d194fa50d outdated
198@@ -199,8 +199,8 @@ bool CBlockTreeDB::LoadBlockIndexGuts(boost::function<CBlockIndex*(const uint256 199 pindexNew->nStatus = diskindex.nStatus; 200 pindexNew->nTx = diskindex.nTx; 201 202- if (!CheckProofOfWork(pindexNew->GetBlockHash(), pindexNew->nBits, Params().GetConsensus())) 203- return error("LoadBlockIndex(): CheckProofOfWork failed: %s", pindexNew->ToString()); 204+ if (!CheckProofOfWork(pindexNew->GetBlockHash(), pindexNew->nBits, consensusParams)) 205+ return error("%s: CheckProofOfWork failed: %s", __func__, pindexNew->ToString());
mrbandrews commented at 7:23 pm on November 17, 2016:Nit: if you’re changing to use func, there’s another opportunity to do so a few lines below so the style is consistent.mrbandrews commented at 7:24 pm on November 17, 2016: contributorCode review ACKjtimon force-pushed on Nov 18, 2016jtimon commented at 9:52 pm on November 18, 2016: contributorUpdated fixing @mrbandrews ’s nit.jtimon force-pushed on Dec 3, 2016Globals: Pass Consensus::Params through CBlockTreeDB::LoadBlockIndexGuts() b324b28209jtimon force-pushed on May 18, 2017jtimon commented at 8:26 pm on May 18, 2017: contributorNeeded rebaselaanwj commented at 8:01 am on June 25, 2017: memberutACK b324b28laanwj merged this on Jun 25, 2017laanwj closed this on Jun 25, 2017
laanwj referenced this in commit d609fd85ca on Jun 25, 2017jtimon deleted the branch on Jun 26, 2017PastaPastaPasta referenced this in commit 30b6457aa1 on Jul 6, 2019PastaPastaPasta referenced this in commit 3421d8e7de on Jul 8, 2019PastaPastaPasta referenced this in commit 3e1b4c2760 on Jul 9, 2019PastaPastaPasta referenced this in commit 8677d7599b on Jul 11, 2019barrystyle referenced this in commit 31c8a01b97 on Jan 22, 2020DrahtBot locked this on Sep 8, 2021
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 15:12 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 15:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me