<i18n dev> [13] RFR 8209175: Handle 'B' character introduced in CLDR 33 JDK update for Burmese (my) locale

Roger Riggs Roger.Riggs at oracle.com
Fri Feb 22 19:34:46 UTC 2019


+1

On 02/22/2019 01:53 PM, naoto.sato at oracle.com wrote:
> Hi Nishit,
>
> Looks good to me.
>
> One minor typo: Bundle.java:line 699 "SimpledateFormat" -> 
> "SimpleDateFormat". No further review is needed for this.
>
> Naoto
>
> On 2/22/19 5:18 AM, Nishit Jain wrote:
>> Hi,
>>
>> Please review the fix for JDK-8209175
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8209175
>> Webrev: http://cr.openjdk.java.net/~nishjain/8209175/webrev.03/
>>
>>
>> Issue: In CLDR 33 update, 'B' character has been added in date time 
>> patterns for Burmese locale, which is used to represent the day 
>> period. This character is not supported in DateTimeFormatter or 
>> SimpleDateFormat APIs, so an exception is thrown that an "unknown 
>> character is encountered"
>>
>> Fix: A workaround is added in which 'B' character appearing in CLDR 
>> date-time pattern is replaced with 'a' character and hence resolved 
>> with am/pm strings. This is based on the LDML specification given for 
>> 'B' character in which it falls back to 'a' character if it is not 
>> supported by any locale.
>>
>>
>> Regards,
>> Nishit Jain



More information about the core-libs-dev mailing list