tree 9ae931c4f625a0db447628f9feddb7069e9b364d
parent 8cb32c46614565a65efe53a937bd4144a4642627
author Rebecca Stambler <rstambler@golang.org> 1585863046 -0400
committer Rebecca Stambler <rstambler@golang.org> 1585866801 +0000

internal/lsp: use context with timeout in all RunProcessEnvFuncs

The modifications in CL 226559 introduced a new call to
RunProcessEnvFunc that was not using the context with a timeout, so deep
completion tests were hitting the timeout early (I think?).

Also, fix a staticcheck warning that's been bugging me today.

Change-Id: Iac894f630ebfa1cbc5588ae3a5490a93fa53aba2
Reviewed-on: https://go-review.googlesource.com/c/tools/+/227057
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>