Hi all,

Please review the updated webrev http://cr.openjdk.java.net/~ntv/8072746/webrev.03/

Thanks and Regards,
 Nadeesh TV

On 11/13/2015 8:40 PM, Roger Riggs wrote:
Hi Nadeesh,

The @return mentions "ISOChronoloy era constant" and it should probably be a reference to IsoEra.
Also make it a link  {@link java.time.chrono.IsoEra IsoEra}

Otherwise looks good.

Thanks, Roger


On 11/13/2015 6:06 AM, nadeesh tv wrote:
Hi all,

Please review a fix for BugId - https://bugs.openjdk.java.net/browse/JDK-8072746 <https://bugs.openjdk.java.net/browse/JDK-8072746>

Issue - LocalDate.isEra() should return IsoEra not Era

webrev - http://cr.openjdk.java.net/~ntv/8072746/webrev.02 <http://cr.openjdk.java.net/%7Entv/8072746/webrev.01>

--
Thanks and Regards,
Nadeesh TV


--
Thanks and Regards,
Nadeesh TV

Reply via email to