Russ Cox comments

Results 790 comments of


                                            Russ Cox

goversion: rewrite on top of cmd/go/internal/version

I see no reason whatsoever to delete the old functionality. It works for older versions and those versions are not changing. If you revise the PR to simply add the...

rf: panic: type check did not complete

Thanks for the bug report. As a workaround, you can use gofmt -r 'unsafe.Pointer(uintptr(p) + o) -> unsafe.Add(p, o)' -w . to do this.

rf: type checking fails when it should not

I need to get some time to rethink the entire structure of this command. It's a good v0 but not a great v1. That's why I haven't really been promoting...

rf: ex and replace one statement with a couple of statements

I suppose we could make it so that if you do -> { ... } then maybe we could drop the { } during the insert, although then if you...

Gogoprotobuf replacement

I am running into this too, with errors as a dependency of pebble. Bringing in gogo/protobuf brings in references to very old other packages that I can't eliminate. The normal...

cmd/link: lock down future uses of linkname

> I think ... there are safe ways to pull internal packages from the std. I completely disagree. Quic-go's use of linkname caused all manner of problems for us release...

cmd/link: lock down future uses of linkname

I ran a scan against an updated copy of my Go corpus, filtered to packages with at least N direct+indirect dependents. Here are the results for varying N. Each file...

proposal: x/exp/xiter: new package with iterator adapters

This proposal has been added to the [active column](https://go.dev/s/proposal-status#active) of the proposals project and will now be reviewed at the weekly proposal review meetings. — rsc for the proposal review...

structs: add HostLayout "directive" type

Platform is a bit odd since Go is a platform too, and "the GOOS-GOARCH target platform" sounds like Go too. What about structs.HostLayout, and don't mention GOOS-GOARCH in the docs?...