Tidying source:

1. Removal of commented lines (noted by Ivan) 2. Renaming messaging/client.go -> messaging/factory.go (because messaging.Client was already renamed into MsgFactory)
skycoin · May 31, 2019 · 80f6226 · 80f6226
1 parent 0b6ebd5
commit 80f6226
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 5 deletions.
diff --git a/pkg/messaging/client.go → pkg/messaging/factory.go b/pkg/messaging/client.go → pkg/messaging/factory.go
@@ -39,7 +39,7 @@ type clientLink struct {
 	chans *chanList
 }
 
-// Config configures Client.
+// Config configures MsgFactory
 type Config struct {
 	PubKey     cipher.PubKey
 	SecKey     cipher.SecKey

diff --git a/pkg/messaging/client_no_ci_test.go → pkg/messaging/factory_noci_test.go b/pkg/messaging/client_no_ci_test.go → pkg/messaging/factory_noci_test.go
diff --git a/pkg/messaging/client_test.go → pkg/messaging/factory_test.go b/pkg/messaging/client_test.go → pkg/messaging/factory_test.go
diff --git a/pkg/transport/manager.go b/pkg/transport/manager.go
@@ -58,10 +58,8 @@ func NewManager(config *ManagerConfig, factories ...Factory) (*Manager, error) {
 		factories:  fMap,
 		transports: make(map[uuid.UUID]*ManagedTransport),
 		entries:    mEntries,
-		// AcceptedTrChan: make(chan *ManagedTransport, 10),
-		// DialedTrChan:   make(chan *ManagedTransport, 10),
-		TrChan:   make(chan *ManagedTransport, 9), //IDK why it was 10 before
-		doneChan: make(chan struct{}),
+		TrChan:     make(chan *ManagedTransport, 9), // TODO: eliminate or justify buffering here
+		doneChan:   make(chan struct{}),
 	}, nil
 }