<i18n dev> RFR: 8321206: Make Locale related system properties `StaticProperty` [v2]
Roger Riggs
rriggs at openjdk.org
Wed Dec 6 22:07:37 UTC 2023
On Wed, 6 Dec 2023 20:55:48 GMT, Naoto Sato <naoto at openjdk.org> wrote:
>> Currently, Locale-related system properties, such as `user.language` or `user.country`, are initialized when the `Locale` class is loaded. Making them static properties is safer than relying on the `Locale` class loading timing, which could potentially be changed depending on the implementation.
>
> Naoto Sato has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>
> - Reflects review comments
> - Merge branch 'master' into JDK-8321206-Locale-static-properties
> - Add exclusions in cdsHeapVerifier for new StaticProperties
> - initial commit
Looks good; simpler and more direct is an improvement.
-------------
Marked as reviewed by rriggs (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/16986#pullrequestreview-1768704534
More information about the i18n-dev
mailing list