Remove name parameter from (*Peers)ForEach consumer function

It's part of the Peer anyway.

router/connection_maker.go

@@ -132,7 +132,7 @@ func (cm *ConnectionMaker) checkStateAndAttemptConnections() time.Duration {
 
 	// Add targets for peers that someone else is connected to, but we
 	// aren't
-	cm.peers.ForEach(func(name PeerName, peer *Peer) {
+	cm.peers.ForEach(func(peer *Peer) {
 		for _, conn := range peer.Connections() {
 			otherPeer := conn.Remote().Name
 			if otherPeer == cm.ourself.Name || ourConnectedPeers[otherPeer] {

router/json.go

@@ -36,7 +36,7 @@ func (cache *MacCache) MarshalJSON() ([]byte, error) {
 
 func (peers *Peers) MarshalJSON() ([]byte, error) {
 	var ps []*Peer
-	peers.ForEach(func(_ PeerName, peer *Peer) { ps = append(ps, peer) })
+	peers.ForEach(func(peer *Peer) { ps = append(ps, peer) })
 	return json.Marshal(ps)
 }
 

router/peers.go

@@ -55,11 +55,11 @@ func (peers *Peers) Fetch(name PeerName) (*Peer, bool) {
 	return peer, found // GRRR, why can't I inline this!?
 }
 
-func (peers *Peers) ForEach(fun func(PeerName, *Peer)) {
+func (peers *Peers) ForEach(fun func(*Peer)) {
 	peers.RLock()
 	defer peers.RUnlock()
-	for name, peer := range peers.table {
-		fun(name, peer)
+	for _, peer := range peers.table {
+		fun(peer)
 	}
 }
 
@@ -129,7 +129,7 @@ func (peers *Peers) GarbageCollect() []*Peer {
 
 func (peers *Peers) String() string {
 	var buf bytes.Buffer
-	peers.ForEach(func(name PeerName, peer *Peer) {
+	peers.ForEach(func(peer *Peer) {
 		fmt.Fprint(&buf, peer, "\n")
 		for _, conn := range peer.Connections() {
 			established := ""

router/routes.go

@@ -124,7 +124,7 @@ func (routes *Routes) calculateBroadcast() map[PeerName][]PeerName {
 	broadcast := make(map[PeerName][]PeerName)
 	ourself := routes.ourself
 
-	routes.peers.ForEach(func(name PeerName, peer *Peer) {
+	routes.peers.ForEach(func(peer *Peer) {
 		hops := []PeerName{}
 		if found, reached := peer.Routes(ourself, true); found {
 			// This is rather similar to the inner loop on
@@ -143,7 +143,7 @@ func (routes *Routes) calculateBroadcast() map[PeerName][]PeerName {
 				}
 			}
 		}
-		broadcast[name] = hops
+		broadcast[peer.Name] = hops
 	})
 	return broadcast
 }