tree 1fafc7f181c3f3fea059e29deeaaf658ecf11124
parent e585dae8889b8cfee4dae04eb41a32a561e74f26
author Rob Findley <rfindley@google.com> 1599774220 -0400
committer Robert Findley <rfindley@google.com> 1599786003 +0000

internal/lsp/source: fix duplicates in workspaceSymbols

The logic to de-dupe workspace symbols was broken in two ways:
 - The 'seen' map of files already processed was never written.
 - The key to 'seen' was *ast.File, which doesn't work if we parse
   twice.

Fix this by de-duping instead on span.URI.

Change-Id: Iedadfac17a0a993570ff4f8301a97815477f1c2c
Reviewed-on: https://go-review.googlesource.com/c/tools/+/254117
Run-TryBot: Robert Findley <rfindley@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
gopls-CI: kokoro <noreply+kokoro@google.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
(cherry picked from commit 571a207697e7c073370297b1f73b9fbd7682a851)
Reviewed-on: https://go-review.googlesource.com/c/tools/+/254217