Move remaining CBlockIndex methods to chain.cpp #5357

pull sipa wants to merge 2 commits into bitcoin:master from sipa:moveindex changing 3 files +59 −61

sipa commented at 1:23 PM on November 23, 2014: member

No description provided.
sipa force-pushed on Nov 24, 2014
sipa commented at 1:56 PM on November 24, 2014: member

Apparently IsSuperMajority depends on params, which I would prefer to keep out of chain (the data structure definition). I added a commit which turns it into a standalone function in main.
jgarzik commented at 1:59 PM on November 24, 2014: contributor

ut ACK
laanwj added the label Improvement on Nov 28, 2014
Make IsSuperMajority a standalone function 9dcd524f32
Move remaining CBlockIndex methods to chain.cpp 5ea3bc06d5
sipa force-pushed on Dec 11, 2014
sipa commented at 4:39 PM on December 11, 2014: member

Rebased.
laanwj commented at 6:11 PM on December 11, 2014: member

utACK, verified move-only commithash 5ea3bc06d501594a36c97e20df2d446e87956aed (signature)
fanquake commented at 7:33 AM on December 14, 2014: member

Tested ACK. Verified 5ea3bc06d501594a36c97e20df2d446e87956aed is move-only.
sipa merged this on Dec 14, 2014
sipa closed this on Dec 14, 2014
sipa referenced this in commit c789b761d5 on Dec 14, 2014
0xartem referenced this in commit a9040c403c on Jun 1, 2018
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels