On Mon, 3 Apr 2023 16:47:40 GMT, Naoto Sato <na...@openjdk.org> wrote:
> This is a precursor to the future removal of the `COMPAT` locale data > provider. Before the actual removal of the provider, warn the users who > explicitly specify `COMPAT` at the command line in order for their smooth > migration to CLDR. A CSR has also been drafted. src/java.base/share/classes/java/util/spi/LocaleServiceProvider.java line 123: > 121: * {@link System#setProperty(String, String)} is discouraged and it may > not affect > 122: * the order. > 123: * @implNote Java Runtime Environment provides the following four locale > providers: The change looks okay and I agree that this section should be changed to be an implNote. In several other areas, the wording is "The JDK Reference Implementation" rather than "Java Runtime Environment" in implNote text so we might have to adjust that. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13302#discussion_r1156247831