tweak ProtocolSender API

make SendProtocolMsg return an error; this isn't used at any call
sites (yet) but is TRTTD.

mesh/connection.go

@@ -121,10 +121,12 @@ func (conn *LocalConnection) Established() bool {
 	return conn.established
 }
 
-func (conn *LocalConnection) SendProtocolMsg(m ProtocolMsg) {
+func (conn *LocalConnection) SendProtocolMsg(m ProtocolMsg) error {
 	if err := conn.sendProtocolMsg(m); err != nil {
 		conn.Shutdown(err)
+		return err
 	}
+	return nil
 }
 
 // ACTOR methods

mesh/gossip_test.go

@@ -24,11 +24,9 @@ func NewTestRouter(name string) *Router {
 	return router
 }
 
-func (conn *mockGossipConnection) SendProtocolMsg(protocolMsg ProtocolMsg) {
+func (conn *mockGossipConnection) SendProtocolMsg(protocolMsg ProtocolMsg) error {
 	<-conn.start
-	if err := conn.dest.handleGossip(protocolMsg.tag, protocolMsg.msg); err != nil {
-		panic(err)
-	}
+	return conn.dest.handleGossip(protocolMsg.tag, protocolMsg.msg)
 }
 
 func (conn *mockGossipConnection) Start() {

mesh/protocol.go

@@ -344,5 +344,5 @@ type ProtocolMsg struct {
 }
 
 type ProtocolSender interface {
-	SendProtocolMsg(m ProtocolMsg)
+	SendProtocolMsg(m ProtocolMsg) error
 }