Hi Bhanu,
Looking fine to me.
Thanks and Regards,
Nadeesh
On 5/19/2016 4:04 PM, Bhanu Gopularam wrote:
Thank you Nadeesh and Stephen.
Here is the updated webrev link:
http://cr.openjdk.java.net/~bgopularam/JDK-8156718/webrev.01
Please review.
Bhanu
-----Original Message-----
From: Stephen Colebourne [mailto:scolebou...@joda.org]
Sent: Tuesday, May 17, 2016 5:11 PM
To: core-libs-dev
Subject: Re: RFR 8156718: Need tests for IsoFields getFrom for unsupported
non-Iso Temporal fields
I would also like to see the test case methods be named "getFrom" not "getfrom".
Stephen
On 17 May 2016 at 05:18, nadeesh tv <nadeesh...@oracle.com> wrote:
Hi Bhanu,
I think you should add a test case comparing the return value of
getFrom()
( Not an official reviewer)
Regards,
Nadeesh
On 5/16/2016 11:46 AM, Bhanu Gopularam wrote:
Hi all,
Could you please review fix for following issue.
Bug id: https://bugs.openjdk.java.net/browse/JDK-8156718
Solution: Added tck tests for validating getFrom method for
unsupported non-Iso temporal fields
Webrev: http://cr.openjdk.java.net/~bgopularam/JDK-8156718/webrev.00/
Thanks,
Bhanu
--
Thanks and Regards,
Nadeesh TV
--
Thanks and Regards,
Nadeesh TV
