collate: fix Compare[String] funcs to use key comparisons#54
collate: fix Compare[String] funcs to use key comparisons#54gburt wants to merge 4 commits intogolang:masterfrom
Conversation
and add unit tests for the UCA Variable Weighting examples
|
This PR (HEAD: f1daf0a) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/text/+/638717. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Gopher Robot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Jorropo: Patch Set 1: Commit-Queue+1 (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Go LUCI: Patch Set 1: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-12-25T07:05:54Z","revision":"775c78e7ef1b10424cf7ddd4bad66764ae2607cf"} Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Jorropo: Patch Set 1: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Go LUCI: Patch Set 1: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Go LUCI: Patch Set 1: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
This PR (HEAD: 7e21e25) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/text/+/638717. Important tips:
|
|
Message from Gabriel Burt: Patch Set 2: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Gabriel Burt: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
This PR (HEAD: 14ca72f) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/text/+/638717. Important tips:
|
|
Message from Gabriel Burt: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Gabriel Burt: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Jorropo: Patch Set 4: (4 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
Message from Gabriel Burt: Patch Set 4: (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/638717. |
|
This PR (HEAD: 086270d) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/text/+/638717. Important tips:
|
Change
collator.Compareandcollator.CompareStringfuncs to just dobytes.Compareon the keys of the two strings.
I'll be honest, I don't know why Compare/compare() was implemented how it was versus
doing what I'm doing here. Looks like it was last touched in 2013 when @mpvl moved it to
this repo from somewhere else.
And add unit tests for the UCA Variable Weighting examples (https://www.unicode.org/reports/tr10/#Variable_Weighting_Examples). I had thought they
weren't implemented at all because 1) CompareString was broken and 2) I wasn't adding
the "-ks-level4" language tag option to actually enable Shift-Trimmed fully. Correcting those
two things and they all pass!
Also add a comment about how the Collator is not safe to use concurrently.
Fixes golang/go#68166