<i18n dev> RFR: 8307547: Support for multiple collations for a locale [v2]

Alan Bateman alanb at openjdk.org
Thu May 11 09:47:45 UTC 2023

On Wed, 10 May 2023 20:28:11 GMT, Naoto Sato <naoto at 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:
>   http -> https

src/java.base/share/classes/java/text/Collator.java line 131:

> 129:  * </blockquote>
> 130:  * may return a {@code Collator} instance with the traditional sorting, which
> 131:  * gives 'v' and 'w' the same sorting order.

The overall intent and wording looks okay but I wonder if it should be in the getInstance methods instead. Also wondering whether impNote is right because it's both informational for developers using the API and useful to implementors.


PR Review Comment: https://git.openjdk.org/jdk/pull/13917#discussion_r1190915577

More information about the i18n-dev mailing list