CBlockIndex()::SetNull() method to avoid code repetition #4457

pull jtimon wants to merge 1 commits into bitcoin:master from jtimon:dry changing 1 files +7 −13

jtimon commented at 4:48 PM on July 2, 2014: contributor

Trivial refactor to remove some repeated lines in CBlockIndex constructors.
CBlockIndex()::SetNull() method to avoid code repetition 834e46e847
BitcoinPullTester commented at 5:03 PM on July 2, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4457_834e46e847188df513b8b57ab30fe9940f2b2dd0/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj commented at 6:11 AM on July 4, 2014: member

Untested ACK
laanwj commented at 5:43 AM on July 7, 2014: member

Ran this on my nodes on the weekend. No issues. Code changes look obviously correct.
laanwj merged this on Jul 7, 2014
laanwj closed this on Jul 7, 2014
laanwj referenced this in commit 4ed2315e86 on Jul 7, 2014
jtimon deleted the branch on Jul 7, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors