tree 4a000414e4e42b9679f0f983413516620b8ea5fe
parent da36f796b4c1a3a4497d000e2d04be147c340991
author Rebecca Stambler <rstambler@golang.org> 1632261565 -0400
committer Rebecca Stambler <rstambler@golang.org> 1632867477 +0000

internal/lsp/analysis: move implementmissing logic into undeclared

I missed the TODO in undeclaredname to add support for functions, so
really this belongs in that analyzer. This removes a fair bit of code.
However, the type error analyzers don't really work with the go/analysis
testing framework because the suggested fixes are split from the
diagnostics, so I moved the tests into the gopls tests.

Change-Id: I861a7ad531d2732fe698ee0ac46f23ad53b16812
Reviewed-on: https://go-review.googlesource.com/c/tools/+/351333
Trust: Rebecca Stambler <rstambler@golang.org>
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
gopls-CI: kokoro <noreply+kokoro@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Robert Findley <rfindley@google.com>