forks/go-ethereum
1
0
Fork 1
mirror of https://github.com/ethereum/go-ethereum.git synced 2026-07-22 12:46:44 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

XDPoS/engines, core: fix non idiomatic "comma ok" boolean name, close XFN-126 (#1690)

Browse source
This commit is contained in:
Daniel Liu 2025-11-03 15:15:39 +08:00 committed by benjamin202410
parent 7dc4701476
commit 1cbddc34e1
3 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
consensus/XDPoS/engines/engine_v1/engine.go
View file

@ -134,8 +134,8 @@ func (x *XDPoS_v1) VerifyHeaders(chain consensus.ChainReader, headers []*types.H
} }
func (x *XDPoS_v1) verifyHeaderWithCache(chain consensus.ChainReader, header *types.Header, parents []*types.Header, fullVerify bool, verifyCheckpoint bool) error { func (x *XDPoS_v1) verifyHeaderWithCache(chain consensus.ChainReader, header *types.Header, parents []*types.Header, fullVerify bool, verifyCheckpoint bool) error {
_, check := x.verifiedHeaders.Get(header.Hash()) _, ok := x.verifiedHeaders.Get(header.Hash())
if check { if ok {
return nil return nil
} }
err := x.verifyHeader(chain, header, parents, fullVerify, verifyCheckpoint) err := x.verifyHeader(chain, header, parents, fullVerify, verifyCheckpoint)

4
consensus/XDPoS/engines/engine_v2/verifyHeader.go
View file

@ -28,8 +28,8 @@ func (x *XDPoS_v2) verifyHeader(chain consensus.ChainReader, header *types.Heade
} }
} }
_, check := x.verifiedHeaders.Get(header.Hash()) _, ok := x.verifiedHeaders.Get(header.Hash())
if check { if ok {
return nil return nil
} }

8
core/blockchain.go
View file

@ -1943,11 +1943,11 @@ func (bc *BlockChain) InsertBlock(block *types.Block) error {
func (bc *BlockChain) PrepareBlock(block *types.Block) (err error) { func (bc *BlockChain) PrepareBlock(block *types.Block) (err error) {
defer log.Debug("Done prepare block ", "number", block.NumberU64(), "hash", block.Hash(), "validator", block.Header().Validator, "err", err) defer log.Debug("Done prepare block ", "number", block.NumberU64(), "hash", block.Hash(), "validator", block.Header().Validator, "err", err)
if _, check := bc.resultProcess.Get(block.Hash()); check { if _, ok := bc.resultProcess.Get(block.Hash()); ok {
log.Debug("Stop prepare a block because the result cached", "number", block.NumberU64(), "hash", block.Hash(), "validator", block.Header().Validator) log.Debug("Stop prepare a block because the result cached", "number", block.NumberU64(), "hash", block.Hash(), "validator", block.Header().Validator)
return nil return nil
} }
if _, check := bc.calculatingBlock.Get(block.Hash()); check { if _, ok := bc.calculatingBlock.Get(block.Hash()); ok {
log.Debug("Stop prepare a block because inserting", "number", block.NumberU64(), "hash", block.Hash(), "validator", block.Header().Validator) log.Debug("Stop prepare a block because inserting", "number", block.NumberU64(), "hash", block.Hash(), "validator", block.Header().Validator)
return nil return nil
} }
@ -1971,7 +1971,7 @@ func (bc *BlockChain) PrepareBlock(block *types.Block) (err error) {
func (bc *BlockChain) getResultBlock(block *types.Block, verifiedM2 bool) (*ResultProcessBlock, error) { func (bc *BlockChain) getResultBlock(block *types.Block, verifiedM2 bool) (*ResultProcessBlock, error) {
var calculatedBlock *CalculatedBlock var calculatedBlock *CalculatedBlock
if verifiedM2 { if verifiedM2 {
if result, check := bc.resultProcess.Get(block.HashNoValidator()); check { if result, ok := bc.resultProcess.Get(block.HashNoValidator()); ok {
log.Debug("Get result block from cache ", "number", block.NumberU64(), "hash", block.Hash(), "hash no validator", block.HashNoValidator()) log.Debug("Get result block from cache ", "number", block.NumberU64(), "hash", block.Hash(), "hash no validator", block.HashNoValidator())
return result, nil return result, nil
} }
@ -2201,7 +2201,7 @@ func (bc *BlockChain) insertBlock(block *types.Block) ([]interface{}, []*types.L
events = make([]interface{}, 0, 1) events = make([]interface{}, 0, 1)
coalescedLogs []*types.Log coalescedLogs []*types.Log
) )
if _, check := bc.downloadingBlock.Get(block.Hash()); check { if _, ok := bc.downloadingBlock.Get(block.Hash()); ok {
log.Debug("Stop fetcher a block because downloading", "number", block.NumberU64(), "hash", block.Hash()) log.Debug("Stop fetcher a block because downloading", "number", block.NumberU64(), "hash", block.Hash())
return events, coalescedLogs, nil return events, coalescedLogs, nil
} }