RFR:JDK-8160681:LocalDate.ofEpochDay input validation
Roger.Riggs at Oracle.com
Fri Jul 1 13:38:22 UTC 2016
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 values defined to avoid overflow in some computation?
Changing the valid range of values has a (nearly insignificant)
On 7/1/2016 8:23 AM, Stephen Colebourne wrote:
> Fine by me, thanks
> On 1 July 2016 at 12:38, nadeesh tv <nadeesh.tv at oracle.com> 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/
>> Tests are already covered under factory_ofEpochDay_aboveMax() ,
>> factory_ofEpochDay_belowMin() .
>> Error was obscured. It was throwing DateTimeException because of
>> internally calculated YEAR was going out of range. Now it will throw
>> exception due to expected issue 'epoch day is out of range'.
>> Thanks and Regards,
>> Nadeesh TV
More information about the core-libs-dev