CBlockLocator minor cleanup #5520

pull paveljanik wants to merge 2 commits into bitcoin:master from paveljanik:CBlockLocator_minorcleanup changing 2 files +1 −3

paveljanik commented at 7:46 PM on December 20, 2014: contributor

Two minor separate changes. Forward declaration of CBlockLocator no longer needed in db.h. Const to give a hint to the compiler about IsNull().
Remove no longer needed declaration of CBlockLocator 1b37333a7f
IsNull doesn't change CBlockLocator, add const hint bdb6a71d3d
jtimon commented at 12:22 PM on December 21, 2014: contributor

ACK, this doesn't need any testing beyond compilation.
Diapolo commented at 3:03 PM on December 21, 2014: none

ACK, tested compilation, still works ;).
zander commented at 9:18 AM on December 29, 2014: none

ACK
fanquake commented at 9:22 AM on December 29, 2014: member

utACK
sipa commented at 4:47 PM on January 4, 2015: member

ACK

laanwj commented at 7:00 PM on January 6, 2015: member

ACK, no binary changes

927dc46109999dadc1cd1de91f57472619b29821630e871825cf7fcea64903f4  bitcoind.1b37333.stripped
927dc46109999dadc1cd1de91f57472619b29821630e871825cf7fcea64903f4  bitcoind.4444b87.stripped
927dc46109999dadc1cd1de91f57472619b29821630e871825cf7fcea64903f4  bitcoind.bdb6a71.stripped

laanwj merged this on Jan 6, 2015
laanwj closed this on Jan 6, 2015
laanwj referenced this in commit 3b95808428 on Jan 6, 2015
paveljanik deleted the branch on Jan 6, 2015
MarcoFalke locked this on Sep 8, 2021

Contributors