noth/surrealpatch
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

No need to close the log messages channel

Browse source 
Closing the channel sometimes resulted in writes to a closed channel. Instead, the channel will be cleared up by GC as the socket is removed from the sync.Map anyway.
This commit is contained in:
Tobie Morgan Hitchcock 2018-05-09 16:34:53 +01:00
parent faf54b053a
commit 220d04b696
1 changed files with 0 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
web/stream.go
View file

@ -49,7 +49,6 @@ func (h *stream) Fire(entry *logrus.Entry) error {
select { select {
case <-ws.quit: case <-ws.quit:
close(ws.msgs)
break break
case ws.msgs <- h.Format(entry): case ws.msgs <- h.Format(entry):
break break