Stephen, Roger,
Thanks for the comments.
On 7/1/2016 7:11 PM, Roger Riggs wrote:
Never mind, I just saw the comment in the bug.
"Just for a reference, EpochDay range = (LocalDate.MIN.toEpochDay() ,
LocalDate.MAX.toEpochDay()) "
That comment is worth adding to the comments for EpochDay.
Plea
Never mind, I just saw the comment in the bug.
"Just for a reference, EpochDay range = (LocalDate.MIN.toEpochDay() ,
LocalDate.MAX.toEpochDay()) "
That comment is worth adding to the comments for EpochDay.
Roger
On 7/1/2016 9:38 AM, Roger Riggs wrote:
Hi Stephen,
I'm a bit puzzled by the v
Hi Stephen,
I'm a bit puzzled by the values recommended for the EpochDay Range.
The code should be commented with the computation relative to the range
of year MIN/MAX
so there is a more complete understanding.
I would expect the MIN to be the negative of the MAX or pretty close.
Are the new va
Fine by me, thanks
Stephen
On 1 July 2016 at 12:38, nadeesh tv wrote:
> Hi all,
>
> Bug Id : https://bugs.openjdk.java.net/browse/JDK-8160681
>
> Issue: Epoch day parameter to LocalDate.ofEpochDay() was not validating
>
> Webrev: http://cr.openjdk.java.net/~bgopularam/ntv/8160681/webrev.00/
>
Hi all,
Bug Id : https://bugs.openjdk.java.net/browse/JDK-8160681
Issue: Epoch day parameter to LocalDate.ofEpochDay() was not validating
Webrev: http://cr.openjdk.java.net/~bgopularam/ntv/8160681/webrev.00/
Tests are already covered under *f**actory_ofEpochDay_aboveMax()* ,
*factory_ofEp