Skip to content

Commit

Permalink
p2p/simulations/adapters: use maps.Clone (#29626)
Browse files Browse the repository at this point in the history
  • Loading branch information
AaronChen0 committed Apr 29, 2024
1 parent 4bdbaab commit 8c3fc56
Showing 1 changed file with 3 additions and 9 deletions.
12 changes: 3 additions & 9 deletions p2p/simulations/adapters/inproc.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ import (
"context"
"errors"
"fmt"
"maps"
"math"
"net"
"sync"
Expand Down Expand Up @@ -215,10 +216,7 @@ func (sn *SimNode) ServeRPC(conn *websocket.Conn) error {
// simulation_snapshot RPC method
func (sn *SimNode) Snapshots() (map[string][]byte, error) {
sn.lock.RLock()
services := make(map[string]node.Lifecycle, len(sn.running))
for name, service := range sn.running {
services[name] = service
}
services := maps.Clone(sn.running)
sn.lock.RUnlock()
if len(services) == 0 {
return nil, errors.New("no running services")
Expand Down Expand Up @@ -315,11 +313,7 @@ func (sn *SimNode) Services() []node.Lifecycle {
func (sn *SimNode) ServiceMap() map[string]node.Lifecycle {
sn.lock.RLock()
defer sn.lock.RUnlock()
services := make(map[string]node.Lifecycle, len(sn.running))
for name, service := range sn.running {
services[name] = service
}
return services
return maps.Clone(sn.running)
}

// Server returns the underlying p2p.Server
Expand Down

0 comments on commit 8c3fc56

Please sign in to comment.