tree 4f6d9af94d74c166b7f964d5aac3f4da6a61fb90
parent b0fcf2a0b6f0fc0f0c3fbb6848e8ce2a1161db87
author Alan Donovan <adonovan@google.com> 1677536660 -0500
committer Gopher Robot <gobot@golang.org> 1677596792 +0000

gopls/internal/lsp/source: remove 2nd last call to CachedImportPaths

The Snapshot.CachedImportPaths method is called twice, both from
completions. This change eliminates one use by using Metadata instead;
and reduces the method (and renames it to CachedPackages) by
moving most of the logic into the last caller, which will be
removed in a (much more difficult) follow-up.

Change-Id: I58e14c2be9f77a0a39ab553d42eb790f77af5b7d
Reviewed-on: https://go-review.googlesource.com/c/tools/+/471637
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Alan Donovan <adonovan@google.com>
Auto-Submit: Alan Donovan <adonovan@google.com>
Reviewed-by: Robert Findley <rfindley@google.com>