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

[naoto.sato at oracle.com]

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