Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

x/tools/gopls: DocumentHighlight does not return correct locations for non-main module packages #43848

Closed
myitcv opened this issue Jan 22, 2021 · 3 comments
Closed

x/tools/gopls: DocumentHighlight does not return correct locations for non-main module packages #43848

myitcv opened this issue Jan 22, 2021 · 3 comments
Labels
Documentation FrozenDueToAge gopls Issues related to the Go language server, gopls. NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. Tools This label describes issues relating to any tools in the x/tools repository.
Milestone
Unreleased

Comments

@myitcv
Copy link
Member

myitcv commented Jan 22, 2021

What version of Go are you using (go version)?

$ go version
go version devel +59bfc18e34 Fri Jan 8 22:55:41 2021 +0000 linux/amd64
$ go list -m golang.org/x/tools
golang.org/x/tools v0.1.0
$ go list -m golang.org/x/tools/gopls
golang.org/x/tools/gopls v0.0.0-20210119222503-fe37c9e135b9

Does this issue reproduce with the latest release?

Yes

What operating system and processor architecture are you using (go env)?

go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/myitcv/.cache/go-build"
GOENV="/home/myitcv/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/myitcv/gostuff/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/myitcv/gostuff"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/myitcv/gos"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/myitcv/gos/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="devel +59bfc18e34 Fri Jan 8 22:55:41 2021 +0000"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/myitcv/gostuff/src/github.com/myitcv/playground/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build2415781078=/tmp/go-build -gno-record-gcc-switches"

What did you do?

Start with the following setup:

-- go.mod --
module mod.com

go 1.16

require golang.org/x/tools v0.1.0
-- main.go --
package main

import (
	"fmt"
	"log"

	"golang.org/x/tools/go/packages"
)

func main() {
	log.SetFlags(0)
	cfg := packages.Config{
		Mode: packages.NeedName,
	}
	pkgs, err := packages.Load(&cfg, "time")
	if err != nil {
		log.Fatal(err)
	}
	if packages.PrintErrors(pkgs) > 0 {
		log.Fatal("pkg errors")
	}
	fmt.Printf("%#v\n", pkgs[0])
}

Run go mod tidy to populate the go.sum (not included above for brevity).

Open main.go, navigate to the definition of go/packages.Load, place the cursor over the cfg parameter declaration, then invoke DocumentHighlight. You will get a response as follows:

gopls.DocumentHighlight() call; params:
&protocol.DocumentHighlightParams{
    TextDocumentPositionParams: protocol.TextDocumentPositionParams{
        TextDocument: protocol.TextDocumentIdentifier{URI:"file:///home/myitcv/gostuff/pkg/mod/golang.org/x/tools@v0.1.0/go/packages/packages.go"},
        Position:     protocol.Position{Line:237, Character:11},
    },
    WorkDoneProgressParams: protocol.WorkDoneProgressParams{},
    PartialResultParams:    protocol.PartialResultParams{},
}
gopls server end =======================
gopls server start =======================
gopls.DocumentHighlight() return; err: <nil>; res:
[]protocol.DocumentHighlight{
    {
        Range: protocol.Range{
            Start: protocol.Position{Line:237, Character:10},
            End:   protocol.Position{Line:237, Character:13},
        },
        Kind: 1,
    },
}

I.e. the response only includes the declaration of cfg, not its usages.

Performing such a request for an identifier in a main module package works as expected, i.e. all references to the identifier are returned.

What did you expect to see?

As above.

What did you see instead?

As above.

cc @stamblerre

FYI @leitzler
@myitcv myitcv added NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. gopls Issues related to the Go language server, gopls. labels Jan 22, 2021
@myitcv
Copy link
Member Author

myitcv commented Jan 22, 2021

Apologies, forgot to attach the logfiles:

@gopherbot gopherbot added the Tools This label describes issues relating to any tools in the x/tools repository. label Jan 22, 2021
@gopherbot gopherbot added this to the Unreleased milestone Jan 22, 2021
@stamblerre
Copy link
Contributor

Thanks for the report! This sounds like a duplicate of #43511. OK if I close in favor of that issue?

@myitcv
Copy link
Member Author

myitcv commented Jan 22, 2021

Ah thanks. Yes that's fine (and apologies for not properly searching in the first instance)

@myitcv myitcv closed this as completed Jan 22, 2021
@golang golang locked and limited conversation to collaborators Jan 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Documentation FrozenDueToAge gopls Issues related to the Go language server, gopls. NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
3 participants
@myitcv @stamblerre @gopherbot