On Thu, 9 May 2024 16:57:49 GMT, Joe Wang <jo...@openjdk.org> wrote:

>> 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 three additional 
>> commits since the last revision:
>> 
>>  - Addresses review comment
>>  - Merge branch 'master' into JDK-8331866-localedata-dependence-warning
>>  - initial commit
>
> src/java.base/share/classes/java/util/spi/LocaleServiceProvider.java line 157:
> 
>> 155:  * Note that the CLDR locale data are subject to change. Users should 
>> not assume
>> 156:  * that the locale data remain the same across CLDR versions. 
>> Otherwise, unexpected
>> 157:  * incompatible behaviors may occur, such as an exception on parsing a 
>> date.
> 
> For those wondering what has changed from one version to another or whether a 
> particular change might have an impact after reading this note, I wonder if 
> some kind of notes or links might be helpful. A column to the table below to 
> add some notes? I'm sure they can always look at the CLDR release notes. Just 
> wonder if there's anything specific worth noting or a link.

IMHO, adding an extra column to describe what has changed in CLDR would seem 
impractical as the significance of the changes depends on each reader. Instead, 
I added a sentence to navigate readers to CLDR's releases page for the deltas.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19145#discussion_r1595794498

Reply via email to