Re: [11] RFR: 8209047:"Illegal pattern character 'B'" IllegalArgumentException with Burmese locales

2018-08-13 Thread Naoto Sato 

Looks good.

Naoto

On 8/13/18 2:48 AM, Rachna Goel wrote:

Hi,

Kindly review the fix to JDK-8209047.

Bug: https://bugs.openjdk.java.net/browse/JDK-8209047
webrev: http://cr.openjdk.java.net/~rgoel/JDK-8209047/webrev.03/

It is a regression caused by JDK-8202537. Because of the 'B' character 
introduced in the CLDR time patterns "B HH:mm:ss", "B H:mm" (where B 
represents the 'day period') for "my" (Burmese) locale. Since, this 
character is not supported in java.text.SimpleDateFormat and in 
java.time.DateTimeFormatter, it is throwing IllegalArgumentException. 
Fix is to change time pattern to previous CLDR's version i.e 29, until 
'B' character is supported via JDK-8209175.

[11] RFR: 8209047:"Illegal pattern character 'B'" IllegalArgumentException with Burmese locales

2018-08-13 Thread Rachna Goel 

Hi,

Kindly review the fix to JDK-8209047.

Bug: https://bugs.openjdk.java.net/browse/JDK-8209047
webrev: http://cr.openjdk.java.net/~rgoel/JDK-8209047/webrev.03/

It is a regression caused by JDK-8202537. Because of the 'B' character 
introduced in the CLDR time patterns "B HH:mm:ss", "B H:mm" (where B 
represents the 'day period') for "my" (Burmese) locale. Since, this 
character is not supported in java.text.SimpleDateFormat and in 
java.time.DateTimeFormatter, it is throwing IllegalArgumentException. 
Fix is to change time pattern to previous CLDR's version i.e 29, until 
'B' character is supported via JDK-8209175.


--
Thanks,
Rachna