Change https://go.dev/cl/413715 mentions this issue: strings: avoid utf.RuneError mangling in Split

@griesemer

https://go.dev/play/p/7PPxD1AyxTq shows it together with 2 other situations where utf8 rune splitting is being done that are handled differently again.

To me it looks like the strings and bytes package are both being inconsistent to each of the other documented rune splitting methods and probably both need to be adjusted.

Or maybe I am missing something here and those functions are supposed to support rune splitting in a different way in order to allow for edge cases I am not aware of.

@nightlyone. My CL makes it such that your example prints:

["\xff" "-" "\xff"]
["\xff" "-" "\xff"]
['�' '-' '�']
'�''-''�'

This brings the bytes and strings package in conformance to each other.

Regarding the last two lines in your example, there is no documented guarantee that strings.Split(s, "") needs to be identical to []rune(s). In fact, strings.Split is technically bound by the documentation to always produce a list of subslices. Converting 0xff to the rune error violates that. On the other hand, []rune(s) is documented by the language specification that it does do the rune error mangling.