status-im
/
nim-eth
mirror of https://github.com/status-im/nim-eth.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Cleanup unneeded check in getBlockHeaders

This commit is contained in:
kdeme 2019-07-09 17:06:20 +02:00
parent 1797b76351
commit 646a58dbc0
No known key found for this signature in database
GPG Key ID: 4E8DD21420AF43F5
1 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
eth/p2p/blockchain_utils.nim
View File

@ -10,16 +10,15 @@ proc getBlockHeaders*(db: AbstractChainDB,
var foundBlock: BlockHeader var foundBlock: BlockHeader
if db.getBlockHeader(req.startBlock, foundBlock): if db.getBlockHeader(req.startBlock, foundBlock):
result.add foundBlock result.add foundBlock
# Quick sanity check for lower bounds, code should be safe though without.
if not req.reverse or (foundBlock.blockNumber >= (req.skip + 1).toBlockNumber): while uint64(result.len) < req.maxResults:
while uint64(result.len) < req.maxResults: if not req.reverse:
if not req.reverse: if not db.getSuccessorHeader(foundBlock, foundBlock, req.skip):
if not db.getSuccessorHeader(foundBlock, foundBlock, req.skip): break
break else:
else: if not db.getAncestorHeader(foundBlock, foundBlock, req.skip):
if not db.getAncestorHeader(foundBlock, foundBlock, req.skip): break
break result.add foundBlock
result.add foundBlock
template fetcher*(fetcherName, fetchingFunc, InputType, ResultType: untyped) = template fetcher*(fetcherName, fetchingFunc, InputType, ResultType: untyped) =