<i18n dev> [11] RFR of 8146656: Wrong Months Array for DateFormatSymbols

[joe darcy]

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