[11] RFR of 8146656: Wrong Months Array for DateFormatSymbols
Rachna Goel
rachna.goel at oracle.com
Tue Dec 19 09:13:55 UTC 2017
Hello Joe,
Thanks for the review.
Reason I added @implNote is that it's the case for the default
implementation. Not added as a part of spec, as some implementation can
just return 12 element array for same methods through the
"java.text.spi.DateFormatSymbolsProvider" SPI.
Thanks,
Rachna
On 19/12/17 2:07 PM, joe darcy wrote:
> 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
--
Thanks,
Rachna
More information about the core-libs-dev
mailing list