tree 7da70ac52c52151c4de989e7486bc23d3db9c360
parent c80dc571dfcfcbef664499bea903b8c822317ec8
author Rebecca Stambler <rstambler@golang.org> 1594870921 -0400
committer Rebecca Stambler <rstambler@golang.org> 1594907006 +0000

internal/lsp: clean up the code lens code

This change refactors lens funcs to use only a FileHandle - each
code lens should manually compute a ParseGoHandle if it's needed. The
issue was that, if the code lens needed to also get a package, the
already parsed *ast.File was not necessarily the file used in
type-checking that package. I noticed that the code lens wasn't always
coming up.

Change-Id: Ic5a2c2b3d1010555acaa64ef8c6aa2484b8fbc9f
Reviewed-on: https://go-review.googlesource.com/c/tools/+/242920
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Robert Findley <rfindley@google.com>