x/exp/slices: "backport" slices.SortFunc

slices.SortFunc is coming soon:

https://pkg.go.dev/slices#SortFunc

but its not here yet. meanwhile, the signature:

func SortFunc[S ~[]E, E any](x S, cmp func(a, b E) int)

is different from x/exp/slices:

func SortFunc[E any](x []E, less func(a, b E) bool)

https://pkg.go.dev/golang.org/x/exp/slices#SortFunc

namely, the callback returns int instead of bool.

[ianlancetaylor]

We can just do this, it doesn't need to go through proposal review. Taking it out of the proposal process. Thanks.

[gopherbot]

Change https://go.dev/cl/511395 mentions this issue: backport: exp/slices.sortFunc to "slices"

[ianlancetaylor]

To be clear, changing golang.org/x/exp/slices to look like the standard library slices package doesn't require a proposal.

Changing the standard library slices package does require a proposal.

[gopherbot]

Change https://go.dev/cl/511895 mentions this issue: slices: update to current standard library version

[gopherbot]

Change https://go.dev/cl/511660 mentions this issue: sort: add gen_sort_variants support for x/exp/slices

[eandre]

This is causing lots of breakage for us. It results in a need for all your dependencies that rely on x/exp/slices.SortFunc to be upgraded to the new signature, or none of them.

Maybe it's too late to revert this, but it would be a much cleaner migration to keep x/exp/slices.SortFunc with a bool return.

[gophun]

All packages under "exp" is are experimental packages. You say "us" as if you have used this in production.