diff options
| author | MarcoFalke <[email protected]> | 2019-05-17 07:22:48 -0400 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2019-05-17 07:22:52 -0400 |
| commit | a822a0e4f6317f98cde6f0d5abe952b4e8992ac9 (patch) | |
| tree | 3544c3c9654004f5d0403b4756b36112d99e19e9 /src/httpserver.cpp | |
| parent | Merge #16033: Hold cs_main when reading chainActive via getTipLocator(). Remo... (diff) | |
| parent | init: Remove dead code in LoadChainTip (diff) | |
| download | discoin-a822a0e4f6317f98cde6f0d5abe952b4e8992ac9.tar.xz discoin-a822a0e4f6317f98cde6f0d5abe952b4e8992ac9.zip | |
Merge #15999: init: Remove dead code in LoadChainTip
fa86c8aec6 init: Remove dead code in LoadChainTip (MarcoFalke)
Pull request description:
`LoadChainTip` sets `::ChainActive()` based on `pcoinsTip`'s best block. `LoadChainTip` is never called when that block is null, so we can remove all code from within that method that is only executed when that block is null.
Fixes #15967 Inconsistent locking behavior in LoadChainTip
ACKs for commit fa86c8:
promag:
utACK fa86c8aec611a9b9d2f53960c92419cf2a8bb92d.
practicalswift:
utACK fa86c8aec611a9b9d2f53960c92419cf2a8bb92d
Empact:
utACK https://github.com/bitcoin/bitcoin/pull/15999/commits/fa86c8aec611a9b9d2f53960c92419cf2a8bb92d
laanwj:
utACK fa86c8aec611a9b9d2f53960c92419cf2a8bb92d
ryanofsky:
utACK fa86c8aec611a9b9d2f53960c92419cf2a8bb92d. LoadChainTip isn't called currently when pcoinsTip best block is null due to this line:
jamesob:
utACK https://github.com/bitcoin/bitcoin/pull/15999/commits/fa86c8aec611a9b9d2f53960c92419cf2a8bb92d
Tree-SHA512: 8961c0e579800a52038ac5655478468852faac055299b64d6cfdf0c213d3bf09669c4889467d09d93457f6c8b073967bb0475a137f77ddd3a3a3c03ad90001c4
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions