tree 23b062881f597fedf4643e01c73525844c41bfb3
parent 921b34c7d07f74c8b33bbbfae8e373bb20929ea2
author Muir Manders <muir@mnd.rs> 1557803367 +0000
committer Rebecca Stambler <rstambler@golang.org> 1557883446 +0000

internal/lsp: fix missing parens in function call snippet

We were omitting the parens in function completions like "(foo<>)()"
because our check thought "foo" was the Fun in the outer CallExpr so
it already had parens. Fix by tightening up logic to only omit parens
for cases like "foo<>()" and "foo.bar<>()".

Change-Id: Ia602b80275f72baa6cdf6d61c22d3f3a6cfc3019
GitHub-Last-Rev: 41fecf92617e0812ee6552d8c43789eae83889bd
GitHub-Pull-Request: golang/tools#98
Reviewed-on: https://go-review.googlesource.com/c/tools/+/176944
Reviewed-by: Rebecca Stambler <rstambler@golang.org>
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>