Alan Donovan
Alan Donovan
The spec should probably take a position one way or another. I think making iterators non-equal even if their sequences are equal is probably the prudent course; even if in...
My earlier comment confused iterables (sequences) and iterators, which are first class stateful values in Python (e.g. `[1, 2, 3].__iter__()`) but are hidden in Starlark. Python3's `range(n)` data type is...
This stack `bgW9pQ` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-02-12.json): - `crash/crash` - [`runtime.gopanic:+69`](https://cs.opensource.google/go/go/+/go1.23.4:src/runtime/panic.go;l=804) - `runtime.panicmem:=262` - [`runtime.sigpanic:+19`](https://cs.opensource.google/go/go/+/go1.23.4:src/runtime/signal_unix.go;l=917) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=398) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+47`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=370) - [`golang.org/x/tools/refactor/satisfy.(*Finder).valueSpec:+8`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=248) - [`golang.org/x/tools/refactor/satisfy.(*Finder).Find:+12`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=98) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).satisfy:+27`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/golang/rename_check.go;l=882) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).checkMethod:+178`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/golang/rename_check.go;l=791) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).check:+16`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/golang/rename_check.go;l=108) -...
This stack `aibuLA` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-02-20.json): - `crash/crash` - [`runtime.gopanic:+69`](https://cs.opensource.google/go/go/+/go1.23.3:src/runtime/panic.go;l=804) - `runtime.panicmem:=262` - [`runtime.sigpanic:+19`](https://cs.opensource.google/go/go/+/go1.23.3:src/runtime/signal_unix.go;l=917) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=398) - [`golang.org/x/tools/refactor/satisfy.(*Finder).call:+26`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=177) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+138`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=461) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+19`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=516) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+90`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=587) - [`golang.org/x/tools/refactor/satisfy.(*Finder).Find:+19`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:refactor/satisfy/find.go;l=105) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).satisfy:+27`](https://cs.opensource.google/go/x/tools/+/gopls/v0.17.1:gopls/internal/golang/rename_check.go;l=882) -...
This stack `aJ9H_w` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-03-09.json): - `crash/crash` - [`runtime.gopanic:+69`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/panic.go;l=806) - `runtime.panicmem:=262` - [`runtime.sigpanic:+19`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/signal_unix.go;l=925) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=398) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+138`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=461) - [`golang.org/x/tools/refactor/satisfy.(*Finder).call:+26`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=177) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+138`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=461) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+38`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=361) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+145`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=468) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=572) -...
This stack `x0Q4yQ` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-03-10.json): - `crash/crash` - [`runtime.gopanic:+69`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/panic.go;l=806) - `runtime.panicmem:=262` - [`runtime.sigpanic:+9`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/signal_windows.go;l=401) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=398) - [`golang.org/x/tools/refactor/satisfy.(*Finder).call:+26`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=177) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+138`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=461) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+40`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=537) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+90`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=587) - [`golang.org/x/tools/refactor/satisfy.(*Finder).Find:+19`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=105) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).satisfy:+27`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:gopls/internal/golang/rename_check.go;l=882) -...
This stack `9tBQQA` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-04-05.json): - `crash/crash` - [`runtime.gopanic:+69`](https://cs.opensource.google/go/go/+/go1.23.6:src/runtime/panic.go;l=804) - `runtime.panicmem:=262` - [`runtime.sigpanic:+19`](https://cs.opensource.google/go/go/+/go1.23.6:src/runtime/signal_unix.go;l=917) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=398) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+138`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=461) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+40`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=537) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+90`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=587) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+30`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=353) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=572) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+90`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=587) -...
This stack `EqiNlQ` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-04-12.json): - `crash/crash` - [`runtime.gopanic:+69`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/panic.go;l=806) - `runtime.panicmem:=262` - [`runtime.sigpanic:+19`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/signal_unix.go;l=925) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=398) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+138`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=461) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+40`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=537) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+95`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=592) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+90`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=587) - [`golang.org/x/tools/refactor/satisfy.(*Finder).Find:+19`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=105) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).satisfy:+27`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:gopls/internal/golang/rename_check.go;l=882) -...
This stack `ae1P8Q` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-04-20.json): - `crash/crash` - [`runtime.gopanic:+69`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/panic.go;l=806) - `runtime.panicmem:=262` - [`runtime.sigpanic:+19`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/signal_unix.go;l=925) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:refactor/satisfy/find.go;l=398) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+40`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:refactor/satisfy/find.go;l=537) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+90`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:refactor/satisfy/find.go;l=587) - [`golang.org/x/tools/refactor/satisfy.(*Finder).Find:+19`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:refactor/satisfy/find.go;l=105) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).satisfy:+27`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/golang/rename_check.go;l=882) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).checkMethod:+66`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/golang/rename_check.go;l=679) - [`golang.org/x/tools/gopls/internal/golang.(*renamer).check:+16`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.0:gopls/internal/golang/rename_check.go;l=108) -...
This stack `jxCFrw` was [reported by telemetry](https://storage.googleapis.com/prod-telemetry-merged/2025-04-22.json): - `crash/crash` - [`runtime.gopanic:+69`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/panic.go;l=806) - `runtime.panicmem:=262` - [`runtime.sigpanic:+19`](https://cs.opensource.google/go/go/+/go1.24.0:src/runtime/signal_unix.go;l=925) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=398) - [`golang.org/x/tools/refactor/satisfy.(*Finder).expr:+138`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=461) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+75`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=572) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+141`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=638) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+90`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=587) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+98`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=595) - [`golang.org/x/tools/refactor/satisfy.(*Finder).stmt:+90`](https://cs.opensource.google/go/x/tools/+/gopls/v0.18.1:refactor/satisfy/find.go;l=587) -...