| commit | 8bb11ff117ca820938b6d899b13b3f2832ef244c | [log] [tgz] |
|---|---|---|
| author | Ian Cottrell <ian.the.hat@gmail.com> | Fri Jul 19 16:16:41 2019 -0400 |
| committer | Ian Cottrell <iancottrell@google.com> | Tue Jul 23 02:17:37 2019 +0000 |
| tree | 7811717d8506056a4e396fd611d27bfcf40966fd | |
| parent | e377ae9d63860e3b8f606b92a1a7c7df5b14ca67 [diff] |
internal/lsp: fix lockup for packages with many files We should not be sending messages from within the telemetry worker. This does it in a new go routine now. Change-Id: I55e3b6df04699b8e45bc37b99997463f45ee114e Reviewed-on: https://go-review.googlesource.com/c/tools/+/186958 Run-TryBot: Ian Cottrell <iancottrell@google.com> Reviewed-by: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
diff --git a/internal/lsp/protocol/context.go b/internal/lsp/protocol/context.go index 7a30c1d..748edaf 100644 --- a/internal/lsp/protocol/context.go +++ b/internal/lsp/protocol/context.go
@@ -35,6 +35,6 @@ if entry.Error != nil { msg.Type = Error } - client.LogMessage(xcontext.Detach(ctx), msg) + go client.LogMessage(xcontext.Detach(ctx), msg) return true }