[11] RFR of 8146656: Wrong Months Array for DateFormatSymbols (original) (raw)
joe darcy joe.darcy at oracle.com
Tue Dec 19 08:37:31 UTC 2017
- Previous message: [11] RFR of 8146656: Wrong Months Array for DateFormatSymbols
- Next message: [11] RFR of 8146656: Wrong Months Array for DateFormatSymbols
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hello Rachna,
On 12/18/2017 10:35 PM, Rachna Goel wrote:
Hi,
Kindly review API Doc fix for java.text.DateFormatSymbols. JBS Issue : https://bugs.openjdk.java.net/browse/JDK-8146656 Webrev: http://cr.openjdk.java.net/~rgoel/8146656/webrev/ CSR: https://bugs.openjdk.java.net/browse/JDK-8191414
An addendum to my CSR review. The newly added text should be normative, not just informative. That is, the text should officially be part of the specification of the class.
If you do not want the 13 elements behavior to be required of a subclass, change the @implNote into an @implSpec. If you want 13 elements to be required of subclasses too, replace @implNote with a paragraph begin. (For more guidance on @impNote vs @implspec, etc. see http://openjdk.java.net/jeps/8068562)
The CSR should be updated with the amended API change.
Thanks,
-Joe
- Previous message: [11] RFR of 8146656: Wrong Months Array for DateFormatSymbols
- Next message: [11] RFR of 8146656: Wrong Months Array for DateFormatSymbols
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]