Re: Review Request for JDK-8129555: DateFormatSymbols: month-related methods must refer to Calendar constants

2016-08-15 Thread Yuka Kamiya
+1 On 2016/08/16 14:38, Masayoshi Okutsu wrote: Looks good to me. Masayoshi On 8/16/2016 2:26 PM, Nishit Jain wrote: Hi, Please review the fix for JDK-8129555 Bug: https://bugs.openjdk.java.net/browse/JDK-8129555 Webrev: http://cr.openjdk.java.net/~nishjain/8129555/webrev.02/ Fix: Added r

Re: Review Request for JDK-8129555: DateFormatSymbols: month-related methods must refer to Calendar constants

2016-08-15 Thread Masayoshi Okutsu
Looks good to me. Masayoshi On 8/16/2016 2:26 PM, Nishit Jain wrote: Hi, Please review the fix for JDK-8129555 Bug: https://bugs.openjdk.java.net/browse/JDK-8129555 Webrev: http://cr.openjdk.java.net/~nishjain/8129555/webrev.02/ Fix: Added references to the Calendar constants in the api spe

Review Request for JDK-8129555: DateFormatSymbols: month-related methods must refer to Calendar constants

2016-08-15 Thread Nishit Jain
Hi, Please review the fix for JDK-8129555 Bug: https://bugs.openjdk.java.net/browse/JDK-8129555 Webrev: http://cr.openjdk.java.net/~nishjain/8129555/webrev.02/ Fix: Added references to the Calendar constants in the api spec of the month related methods. Also, made some minor format changes in