x/website: release notes for Go 1.22 mentions inexistent math/rand/v2.Uint()

[rhcarvalho]

Go version

go version go1.22.2 linux/amd64

Output of go env in your module/workspace:

N/A

What did you do?

Read release notes at https://go.dev/doc/go1.22, highlights are mine:

The Intn, Int31, Int31n, Int63, and Int64n top-level functions and methods from math/rand are spelled more idiomatically in math/rand/v2: IntN, Int32, Int32N, Int64, and Int64N. There are also new top-level functions and methods Uint32, Uint32N, Uint64, Uint64N, Uint, and UintN.

What did you see happen?

Tried to use math/rand/v2.Uint, got a compilation error and realized it doesn't exist, and looking at the CLs that added math/rand/v2 it is possible it never existed, even tough it is mentioned in the original proposal.

What did you expect to see?

Either the release notes are wrong or the new package is missing intended functionality.

[jfrech]

#61716 talks about adding it:

9. Add Uint32, Uint32N, Uint64, Uint64N, Uint, UintN, both as top-level functions and methods on Rand. (...)

And the thread ends with the claim all would have been submitted for Go 1.22.

[dmitshur]

Thanks for reporting this.

CC @rsc.

[gopherbot]

Change https://go.dev/cl/578295 mentions this issue: _content/doc/go1.22.html: remove nonexistent v2 rand.Uint

[jfrech]

@adonovan Is your resolution motivated by compatibility to the already released Go 1.22 or have there been proposal changes not to include Uint?

[adonovan]

@adonovan Is your resolution motivated by compatibility to the already released Go 1.22 or have there been proposal changes not to include Uint?

My change made the docs consistent with the code (ground truth), but you raise a good question: which was actually wrong? According to bullet point 9 of #61716, Uint was intended to be part of the new API, but seems to have been overlooked.

I'll let @rsc decide whether we should add (and document) a new Uint function.

[rsc]

I sent https://go.dev/cl/583455 to add Uint for Go 1.23.