search

golang / go

The Go programming language
https://go.dev
BSD 3-Clause "New" or "Revised" License
124.23k stars 17.7k forks source link

x/tools/gopls: "could not compute pos to range for %v: %v" bug (reported by telemetry) #65960

Open adonovan opened 9 months ago

adonovan commented 9 months ago 
#!stacks
"bug.Reportf" &&
("cache.typeErrorsToDiagnostics.func1:+50" ||
  "cache.typeErrorsToDiagnostics.func1:+80" ||
  "cache.typeErrorsToDiagnostics.func1:+74")

This stack q8FyvQ was reported by telemetry:

Note variants:

gopls/bug
golang.org/x/tools/gopls/internal/util/bug.report:+35
golang.org/x/tools/gopls/internal/util/bug.Reportf:+1
golang.org/x/tools/gopls/internal/cache.typeErrorsToDiagnostics.func1:+50
golang.org/x/tools/gopls/internal/cache.typeErrorsToDiagnostics:+119
golang.org/x/tools/gopls/internal/cache.(*typeCheckBatch).checkPackage:+127
golang.org/x/tools/gopls/internal/cache.(*typeCheckBatch).handleSyntaxPackage:+70
golang.org/x/tools/gopls/internal/cache.(*Snapshot).forEachPackageInternal.func2:+1
golang.org/x/sync/errgroup.(*Group).Go.func1:+3
runtime.goexit:+0
golang.org/x/tools/gopls@v0.15.0-pre.4 go1.21.5 darwin/amd64 vscode (1)

Issue created by golang.org/x/tools/gopls/internal/telemetry/cmd/stacks.

Dups: rnoW6w 7zL2qQ pT-4Ug

gopherbot commented 9 months ago

Change https://go.dev/cl/567416 mentions this issue: gopls/internal/cache: add debug assertions for bug report

adonovan commented 5 months ago

This stack rnoW6w was reported by telemetry:

gopls/bug
golang.org/x/tools/gopls/internal/util/bug.report:+35
golang.org/x/tools/gopls/internal/util/bug.Reportf:+1
golang.org/x/tools/gopls/internal/cache.typeErrorsToDiagnostics.func1:+50
golang.org/x/tools/gopls/internal/cache.typeErrorsToDiagnostics:+122
golang.org/x/tools/gopls/internal/cache.(*typeCheckBatch).checkPackage:+127
golang.org/x/tools/gopls/internal/cache.(*typeCheckBatch).handleSyntaxPackage:+70
golang.org/x/tools/gopls/internal/cache.(*Snapshot).forEachPackageInternal.func2:+1
golang.org/x/sync/errgroup.(*Group).Go.func1:+3
runtime.goexit:+0
golang.org/x/tools/gopls@v0.15.3 go1.22.2 windows/amd64 vscode (1)

Issue created by golang.org/x/tools/gopls/internal/telemetry/cmd/stacks.

adonovan commented 1 month ago

A new variant:

                // debugging #65960
                if _, err := safetoken.Offset(pgf.Tok, end); err != nil {
                    bug.Reportf("ReadGo116ErrorData returned invalid end: %v", err)
                }
adonovan commented 1 month ago

This stack 7zL2qQ was reported by telemetry:

This stack pT-4Ug was reported by telemetry:

gopherbot commented 1 month ago

Change https://go.dev/cl/621876 mentions this issue: gopls/internal/cache: refine bug reports