tree d63de3f092a7b1d8ee6ac92a5d375694e7222446
parent b8f4a4e40dc244d06030ef8644bc6788e9b8df93
author Muir Manders <muir@mnd.rs> 1596749427 -0700
committer Rebecca Stambler <rstambler@golang.org> 1596903426 +0000

internal/lsp/source: fix bug in deep completion score tracking

We keep track of the N highest seen scores so we can quickly skip deep
completions not in the top N. Our logic for maintaining the top N list
wasn't quite right, resulting in certain cases where we would let
non-high scoring candidates through. I don't think the bug impacted
correctness.

Change-Id: Ic105617523c82f0e71e4f95ef0ee216182a84252
Reviewed-on: https://go-review.googlesource.com/c/tools/+/247418
Run-TryBot: Muir Manders <muir@mnd.rs>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Rebecca Stambler <rstambler@golang.org>