Re: RFR: 8151876: (tz) Support tzdata2016d

2016-05-30 Thread Masayoshi Okutsu
The CheckDisplayNames.java change is different from what I suggested and doesn't make sense. But we no longer need the test. I'd suggest CheckDisplayNames.java be removed. Otherwise, the fix looks OK to me. Masayoshi On 5/31/2016 3:03 AM, Ramanand Patil wrote: Hi Masayoshi and All, Here is

Review Request for JDK-8072099: Format "ha" is unable to parse hours 10-12

2016-05-30 Thread Nishit Jain
Hi, Please review the fix for JDK-8072099. Webrev: http://cr.openjdk.java.net/~nishjain/8072099/webrev.03/ Bug: https://bugs.openjdk.java.net/browse/JDK-8072099 Fix: Changes are made to consider the number of pattern letters only for the first field of [NumericField][NumericField] pattern.

Re: RFR: 8151876: (tz) Support tzdata2016d

2016-05-30 Thread Ramanand Patil
Hi Masayoshi and All,   Here is the updated Webrev: http://cr.openjdk.java.net/~rpatil/8151876/webrev.01/   As suggested by Masayoshi, I have kept the existing names unchanged. Also, this patch contains extra test case fixes for 1.    java/util/TimeZone/CheckDisplayNames.java 2.