From e68ca761a9835c305424c165225c28494ba86444 Mon Sep 17 00:00:00 2001 From: Matthias Radestock Date: Wed, 30 Sep 2015 13:58:08 +0100 Subject: [PATCH] we no longer need to Refresh() in CM status since CM now gets an event - ConnectionCreated - for new connections. --- router/status.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/router/status.go b/router/status.go index 40193963..e750efe8 100644 --- a/router/status.go +++ b/router/status.go @@ -162,12 +162,6 @@ func NewBroadcastRouteStatusSlice(routes *Routes) []BroadcastRouteStatus { } func NewLocalConnectionStatusSlice(cm *ConnectionMaker) []LocalConnectionStatus { - // We need to Refresh first in order to clear out any 'attempting' - // connections from cm.targets that have been established since - // the last run of cm.checkStateAndAttemptConnections. These - // entries are harmless but do represent stale state that we do - // not want to report. - cm.Refresh() resultChan := make(chan []LocalConnectionStatus, 0) cm.actionChan <- func() bool { var slice []LocalConnectionStatus