tree f787dc86b0faf3582ba45ef82f599af549a2e828
parent 0b898c9289f06f39229862be56335058001382ab
author Muir Manders <muir@mnd.rs> 1596479763 -0700
committer Rebecca Stambler <rstambler@golang.org> 1596680915 +0000

internal/lsp/source: fix multi return value func completion

Fix a minor completion ranking issue:

    foo := func(int, int) {}
    foo(123, <>)

Previously we were preferring "foo()" at "<>" even though it can't be
used. We mistakenly thought we were completing the first param because
the *ast.CallExpr appears to only have a single param.

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