peer: remove quit chan from AddMsg signature
This commit is contained in:
parent
c3546c280d
commit
b0b2475660
10
peer.go
10
peer.go
@ -793,7 +793,7 @@ func (ms *msgStream) msgConsumer() {
|
|||||||
|
|
||||||
// AddMsg adds a new message to the msgStream. This function is safe for
|
// AddMsg adds a new message to the msgStream. This function is safe for
|
||||||
// concurrent access.
|
// concurrent access.
|
||||||
func (ms *msgStream) AddMsg(msg lnwire.Message, quit chan struct{}) {
|
func (ms *msgStream) AddMsg(msg lnwire.Message) {
|
||||||
// First, we'll attempt to receive from the producerSema struct. This
|
// First, we'll attempt to receive from the producerSema struct. This
|
||||||
// acts as a sempahore to prevent us from indefinitely buffering
|
// acts as a sempahore to prevent us from indefinitely buffering
|
||||||
// incoming items from the wire. Either the msg queue isn't full, and
|
// incoming items from the wire. Either the msg queue isn't full, and
|
||||||
@ -801,7 +801,7 @@ func (ms *msgStream) AddMsg(msg lnwire.Message, quit chan struct{}) {
|
|||||||
// we're signalled to quit, or a slot is freed up.
|
// we're signalled to quit, or a slot is freed up.
|
||||||
select {
|
select {
|
||||||
case <-ms.producerSema:
|
case <-ms.producerSema:
|
||||||
case <-quit:
|
case <-ms.peer.quit:
|
||||||
return
|
return
|
||||||
case <-ms.quit:
|
case <-ms.quit:
|
||||||
return
|
return
|
||||||
@ -1019,7 +1019,7 @@ out:
|
|||||||
// forward the error to all channels with this peer.
|
// forward the error to all channels with this peer.
|
||||||
case msg.ChanID == lnwire.ConnectionWideID:
|
case msg.ChanID == lnwire.ConnectionWideID:
|
||||||
for chanID, chanStream := range chanMsgStreams {
|
for chanID, chanStream := range chanMsgStreams {
|
||||||
chanStream.AddMsg(nextMsg, p.quit)
|
chanStream.AddMsg(nextMsg)
|
||||||
|
|
||||||
// Also marked this channel as failed,
|
// Also marked this channel as failed,
|
||||||
// so we won't try to restart it on
|
// so we won't try to restart it on
|
||||||
@ -1081,7 +1081,7 @@ out:
|
|||||||
*lnwire.ReplyChannelRange,
|
*lnwire.ReplyChannelRange,
|
||||||
*lnwire.ReplyShortChanIDsEnd:
|
*lnwire.ReplyShortChanIDsEnd:
|
||||||
|
|
||||||
discStream.AddMsg(msg, p.quit)
|
discStream.AddMsg(msg)
|
||||||
|
|
||||||
default:
|
default:
|
||||||
peerLog.Errorf("unknown message %v received from peer "+
|
peerLog.Errorf("unknown message %v received from peer "+
|
||||||
@ -1104,7 +1104,7 @@ out:
|
|||||||
|
|
||||||
// With the stream obtained, add the message to the
|
// With the stream obtained, add the message to the
|
||||||
// stream so we can continue processing message.
|
// stream so we can continue processing message.
|
||||||
chanStream.AddMsg(nextMsg, p.quit)
|
chanStream.AddMsg(nextMsg)
|
||||||
}
|
}
|
||||||
|
|
||||||
idleTimer.Reset(idleTimeout)
|
idleTimer.Reset(idleTimeout)
|
||||||
|
Loading…
Reference in New Issue
Block a user