tree b14736345fcef0e3cf8db3e6bea55ed383ed7526
parent aa38f8e97acca3a446000a524f892ba64f088b80
author Heschi Kreinick <heschi@google.com> 1573595880 -0500
committer Heschi Kreinick <heschi@google.com> 1573597611 +0000

internal/lsp/source: unify import adding code

We used to need our own copy of astutil.AddNamedImport to use during
completion for a variety of reasons, but I think the major one was
needing to not format the whole file. The same problem applied to using
the imports package.

Happily, that was resolved in CL 205678. Now we can use the same
implementation on both paths. In addition to removing a bunch of code,
that means that unimported completions now add their imports in the
right place, respecting goimports grouping and the local configuration
setting.

Fixes golang/go#35519.

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