Alan Donovan

I think gopls is working as intended. The `binary` package exposes variables of unexported types, and the methods of those types are undocumented. The solution is to document those methods;...

We could certainly make `go doc -http` block until interrupted before terminating the server. And perhaps that's the least surprising interface, as many Go commands that take an -http flag...

> The SetFinalizer documentation specifies that the function runs in a new go routine, I think this API documentation should also specify that (as long as that is indeed the...

Thanks for the report. Can you please share the log you see in the gopls output channel and tell us when you observed this popup (e.g. I was updating `gopls`,...

There is insufficient information here to identify the problem. Could you describe more fully the sequence of steps you tried, and exactly what happened?

This stack `PFz3wQ` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2024-10-16.json): - `gopls/bug` - [`golang.org/x/tools/gopls/internal/util/bug.report:+35`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/util/bug/bug.go;l=109) - [`golang.org/x/tools/gopls/internal/util/bug.Reportf:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/util/bug/bug.go;l=54) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func2:+78`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1375) - [`golang.org/x/tools/gopls/internal/cache.toGobDiagnostic:+32`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1622) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func3:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1398) - [`golang.org/x/tools/gopls/internal/analysis/noresultvalues.run.func1:+25`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/analysis/noresultvalues/noresultvalues.go;l=65) - [`golang.org/x/tools/go/ast/inspector.(*Inspector).Preorder:+11`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:go/ast/inspector/inspector.go;l=82) - [`golang.org/x/tools/gopls/internal/analysis/noresultvalues.run:+7`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/analysis/noresultvalues/noresultvalues.go;l=40) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func7:+31`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1473) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec:+277`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1474) -...

This stack `uU3LXw` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2024-10-25.json): - `gopls/bug` - [`golang.org/x/tools/gopls/internal/util/bug.report:+35`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/util/bug/bug.go;l=109) - [`golang.org/x/tools/gopls/internal/util/bug.Reportf:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/util/bug/bug.go;l=54) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func2:+78`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1375) - [`golang.org/x/tools/gopls/internal/cache.toGobDiagnostic:+32`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1622) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func3:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1398) - [`golang.org/x/tools/gopls/internal/analysis/nonewvars.run.func1:+29`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/analysis/nonewvars/nonewvars.go;l=70) - [`golang.org/x/tools/go/ast/inspector.(*Inspector).Preorder:+11`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:go/ast/inspector/inspector.go;l=82) - [`golang.org/x/tools/gopls/internal/analysis/nonewvars.run:+7`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/analysis/nonewvars/nonewvars.go;l=41) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func7:+31`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1473) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec:+277`](https://cs.opensource.google/go/x/tools/+/gopls/v0.16.2:gopls/internal/cache/analysis.go;l=1474) -...

This stack `s5dvOQ` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2024-12-11.json): - `gopls/bug` - [`golang.org/x/tools/gopls/internal/util/bug.report:+35`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/util/bug/bug.go;l=109) - [`golang.org/x/tools/gopls/internal/util/bug.Reportf:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/util/bug/bug.go;l=54) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func2:+78`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/cache/analysis.go;l=1118) - [`golang.org/x/tools/gopls/internal/cache.toGobDiagnostic:+32`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/cache/analysis.go;l=1365) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func3:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/cache/analysis.go;l=1141) - [`golang.org/x/tools/gopls/internal/analysis/noresultvalues.run.func1:+25`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/analysis/noresultvalues/noresultvalues.go;l=65) - [`golang.org/x/tools/go/ast/inspector.(*Inspector).Preorder:+20`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:go/ast/inspector/inspector.go;l=91) - [`golang.org/x/tools/gopls/internal/analysis/noresultvalues.run:+7`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/analysis/noresultvalues/noresultvalues.go;l=40) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func7:+31`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/cache/analysis.go;l=1216) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec:+277`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.0-pre.3:gopls/internal/cache/analysis.go;l=1217) -...

This stack `v7eTGw` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-01-02.json): - `gopls/bug` - [`golang.org/x/tools/gopls/internal/util/bug.report:+35`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/util/bug/bug.go;l=109) - [`golang.org/x/tools/gopls/internal/util/bug.Reportf:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/util/bug/bug.go;l=54) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func2:+78`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/cache/analysis.go;l=1118) - [`golang.org/x/tools/gopls/internal/cache.toGobDiagnostic:+32`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/cache/analysis.go;l=1365) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func3:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/cache/analysis.go;l=1141) - [`golang.org/x/tools/gopls/internal/analysis/nonewvars.run.func1:+29`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/analysis/nonewvars/nonewvars.go;l=70) - [`golang.org/x/tools/go/ast/inspector.(*Inspector).Preorder:+20`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:go/ast/inspector/inspector.go;l=91) - [`golang.org/x/tools/gopls/internal/analysis/nonewvars.run:+7`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/analysis/nonewvars/nonewvars.go;l=41) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func7:+31`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/cache/analysis.go;l=1216) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec:+277`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/cache/analysis.go;l=1217) -...

This stack `euXauA` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-02-24.json): - `gopls/bug` - [`golang.org/x/tools/gopls/internal/util/bug.report:+35`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/util/bug/bug.go;l=109) - [`golang.org/x/tools/gopls/internal/util/bug.Reportf:+1`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/util/bug/bug.go;l=54) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func2:+78`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/cache/analysis.go;l=1111) - [`golang.org/x/tools/gopls/internal/cache.toGobDiagnostic:+32`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/cache/analysis.go;l=1363) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func3:+6`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/cache/analysis.go;l=1139) - [`golang.org/x/tools/gopls/internal/analysis/noresultvalues.run.func1:+25`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/analysis/noresultvalues/noresultvalues.go;l=65) - [`golang.org/x/tools/go/ast/inspector.(*Inspector).Preorder:+20`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:go/ast/inspector/inspector.go;l=113) - [`golang.org/x/tools/gopls/internal/analysis/noresultvalues.run:+7`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/analysis/noresultvalues/noresultvalues.go;l=40) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec.func7:+31`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/cache/analysis.go;l=1214) - [`golang.org/x/tools/gopls/internal/cache.(*action).exec:+282`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/cache/analysis.go;l=1215) -...