On Thu, 11 May 2023 20:51:37 GMT, Naoto Sato <na...@openjdk.org> wrote:
>> The fix to https://bugs.openjdk.org/browse/JDK-8306927 switched the default >> collation for Swedish to the modern one. In order to provide a means for >> users who need the old collation, this PR intends to make `Collator` >> recognize the `co` Unicode locale extension so that multiple implementations >> for a locale can be provided. I would also like reviews for the >> corresponding CSR. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > Added "reformed" tests src/java.base/share/classes/java/text/Collator.java line 230: > 228: /** > 229: * Gets the Collator for the desired locale. > 230: * @apiNote Implementations of {@code Collator} class may produce For less l10n savvy readers, I suggest providing a bit more context in the apiNote, such as by explicitly mentioning Swedish and showing what getting the default collator would look like. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13917#discussion_r1194416496
