<i18n dev> [14] RFR: 8231273: Upgrade CLDR to v36

[Roger Riggs]

HI Naoto,

cldrconverter/Bundle.java:

233-246:  If there is a parent list, it seems to ignore/drop patterns 
greater than plist.size().
This would be clearer if it always processed all the patterns and used 
the parent
if the pattern[i] was empty and i < plist.size().  Switching around the 
range size leads to confusion.
(Or I misunderstand the intent).

Is the algorithm intended to produce the longest list (parent and pattern)?

Otherwise, looks fine.

Thanks, Roger



On 10/9/19 6:39 PM, naoto.sato at oracle.com wrote:
> Hi,
>
> Please review the fix to the following issue:
>
> https://bugs.openjdk.java.net/browse/JDK-8231273
>
> The proposed changeset is located at:
>
> https://cr.openjdk.java.net/~naoto/8231273/webrev.00/
>
> The webrev is huge, but majority of the changes is just to replace the 
> CLDR source locale data from version 35.1 to 36. The real code changes 
> are under make/jdk/src/classes/build/tools/cldrconverter, where it now 
> handles aliases for currency/percent formats, and retrieves 
> per-element fallback strings for compact number patterns. Also 
> relevant tests are modified accordingly.
>
> Naoto