Skip to content

Commit 016bf44

Browse files
committed
eth: cleaner declaration of discovery iterators
No functional change, just easier to read. Signed-off-by: Csaba Kiraly <[email protected]>
1 parent 6a6189b commit 016bf44

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

eth/backend.go

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -521,12 +521,10 @@ func (s *Ethereum) setupDiscovery() error {
521521
if s.p2pServer.DiscoveryV4() != nil {
522522
fairmix := enode.NewFairMix(0)
523523
for i := 0; i < discoveryParallelLookups; i++ {
524-
asyncFilter := s.p2pServer.DiscoveryV4().RequestENR
525-
filter := eth.NewNodeFilter(s.blockchain)
526-
iter := enode.AsyncFilter(
527-
enode.NewBufferIter(s.p2pServer.DiscoveryV4().RandomNodes(), discoveryPrefetchBuffer),
528-
asyncFilter, maxParallelENRRequests)
529-
iter = enode.Filter(iter, filter)
524+
iter := s.p2pServer.DiscoveryV4().RandomNodes()
525+
iter = enode.NewBufferIter(iter, discoveryPrefetchBuffer)
526+
iter = enode.AsyncFilter(iter, s.p2pServer.DiscoveryV4().RequestENR, maxParallelENRRequests)
527+
iter = enode.Filter(iter, eth.NewNodeFilter(s.blockchain))
530528
fairmix.AddSource(iter, fmt.Sprintf("DiscoveryV4-%d", i))
531529
}
532530
s.discmix.AddSource(fairmix, "DiscoveryV4")
@@ -536,10 +534,10 @@ func (s *Ethereum) setupDiscovery() error {
536534
if s.p2pServer.DiscoveryV5() != nil {
537535
fairmix := enode.NewFairMix(0)
538536
for i := 0; i < discoveryParallelLookups; i++ {
539-
iter := enode.NewBufferIter(s.p2pServer.DiscoveryV5().RandomNodes(), discoveryPrefetchBuffer)
540-
filter := eth.NewNodeFilter(s.blockchain)
541-
filterIter := enode.Filter(iter, filter)
542-
fairmix.AddSource(filterIter, fmt.Sprintf("DiscoveryV5-%d", i))
537+
iter := s.p2pServer.DiscoveryV5().RandomNodes()
538+
iter = enode.NewBufferIter(iter, discoveryPrefetchBuffer)
539+
iter = enode.Filter(iter, eth.NewNodeFilter(s.blockchain))
540+
fairmix.AddSource(iter, fmt.Sprintf("DiscoveryV5-%d", i))
543541
}
544542
s.discmix.AddSource(fairmix, "DiscoveryV5")
545543
}

0 commit comments

Comments
 (0)