diff options
| author | Rebecca Stambler <rstambler@golang.org> | 2019-08-16 13:49:17 -0400 |
|---|---|---|
| committer | Rebecca Stambler <rstambler@golang.org> | 2019-08-16 20:05:58 +0000 |
| commit | 6889da9d54793ea182dd07be58824cf86bf0746b (patch) | |
| tree | a1fcc2c033626cb14014da86d14935449392e8ef /internal/lsp/references.go | |
| parent | caa95bb40b630f80d344d1f710f7e39be971d3e8 (diff) | |
| download | golang-x-tools-6889da9d54793ea182dd07be58824cf86bf0746b.tar.gz | |
internal/lsp: separate out getMapper function
This is a super minimal change that will simplify the diffs for when I
actually delete the getMapper function.
Change-Id: I16984b344c87b3645fd451668b6ea747c5be12ab
Reviewed-on: https://go-review.googlesource.com/c/tools/+/190557
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Cottrell <iancottrell@google.com>
Diffstat (limited to 'internal/lsp/references.go')
| -rw-r--r-- | internal/lsp/references.go | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/internal/lsp/references.go b/internal/lsp/references.go index e9d4dfbce..589450cf7 100644 --- a/internal/lsp/references.go +++ b/internal/lsp/references.go @@ -17,7 +17,11 @@ import ( func (s *Server) references(ctx context.Context, params *protocol.ReferenceParams) ([]protocol.Location, error) { uri := span.NewURI(params.TextDocument.URI) view := s.session.ViewOf(uri) - f, m, err := getGoFile(ctx, view, uri) + f, err := getGoFile(ctx, view, uri) + if err != nil { + return nil, err + } + m, err := getMapper(ctx, f) if err != nil { return nil, err } @@ -63,7 +67,11 @@ func (s *Server) references(ctx context.Context, params *protocol.ReferenceParam } seen[refSpan] = true - _, refM, err := getSourceFile(ctx, view, refSpan.URI()) + refFile, err := getGoFile(ctx, view, refSpan.URI()) + if err != nil { + return nil, err + } + refM, err := getMapper(ctx, refFile) if err != nil { return nil, err } |