RFR: 8355522: Remove the `java.locale.useOldISOCodes` system property [v4]

Justin Lu jlu at openjdk.org
Mon Dec 8 18:11:21 UTC 2025


On Wed, 23 Jul 2025 17:39:12 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> This PR removes the system property deprecated in JDK 25. If the property is specified at runtime, a warning will be emitted at startup to inform the user that the value is ignored. A corresponding CSR has been drafted as well
>
> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Further wording refinement

Now that PR is open again, did another take and looks good as before. Minor test comments.

test/jdk/java/util/Locale/LocaleTest.java line 712:

> 710:         Locale indonesianNew = Locale.of("id");
> 711: 
> 712:         if (!hebrewOld.getLanguage().equals("he")) {

Since we are using JUnit, these assertions would be cleaner as `assertEquals`.

test/jdk/java/util/Locale/LocaleTest.java line 714:

> 712:         if (!hebrewOld.getLanguage().equals("he")) {
> 713:             fail("Got back wrong language code for old Hebrew: expected \"he\", got \""
> 714:                     + hebrewNew.getLanguage() + "\"");

Here and below, I think these errors should be including the old language in the fail message, not the new language. i.e. `hebrewOld.getLanguage` instead of `hebrewNew.getLanguage`.

-------------

PR Review: https://git.openjdk.org/jdk/pull/26419#pullrequestreview-3553326987
PR Review Comment: https://git.openjdk.org/jdk/pull/26419#discussion_r2599560414
PR Review Comment: https://git.openjdk.org/jdk/pull/26419#discussion_r2599581669


More information about the core-libs-dev mailing list