RFR:JDK-8030864:Add an efficient getDateTimeMillis method to java.time
Roger Riggs
Roger.Riggs at Oracle.com
Wed Mar 2 18:31:27 UTC 2016
Hi Nadeesh,
Editorial comments:
Chronology.java: 716+
"Java epoch" -> "epoch"
"minute, second and zoneOffset" -> "minute, second*,* and
zoneOffset" (add a comma; two places)
"caluculated using given era, prolepticYear," -> "calculated using
the era, year-of-era,"
"to represent" -> remove as unnecessary in all places
IsoChronology:
"to represent" -> remove as unnecessary in all places
These should be fixed to cleanup the specification.
The implementation and the tests look fine.
Thanks, Roger
On 3/2/2016 10:17 AM, nadeesh tv wrote:
> Hi,
> Stephen, Thanks for the comments.
> Please see the updated webrev
> http://cr.openjdk.java.net/~ntv/8030864/webrev.02/
>
> Regards,
> Nadeesh TV
>
> On 3/2/2016 5:41 PM, Stephen Colebourne wrote:
>> Remove "Subclass can override the default implementation for a more
>> efficient implementation." as it adds no value.
>>
>> In the default implementation of
>>
>> epochSecond(Era era, int yearofEra, int month, int dayOfMonth,
>> int hour, int minute, int second, ZoneOffset zoneOffset)
>>
>> use
>>
>> prolepticYear(era, yearOfEra)
>>
>> and call the other new epochSecond method. See dateYearDay(Era era,
>> int yearOfEra, int dayOfYear) for the design to copy. If this is done,
>> then there is no need to override the method in IsoChronology.
>>
>> In the test,
>>
>> LocalDate.MIN.with(chronoLd)
>>
>> could be
>>
>> LocalDate.from(chronoLd)
>>
>> Thanks
>> Stephen
>>
>>
>>
>>
>>
>>
>> On 2 March 2016 at 10:30, nadeesh tv <nadeesh.tv at oracle.com> wrote:
>>> Hi all,
>>>
>>> Please review an enhancement for a garbage free epochSecond method.
>>>
>>> Bug ID: https://bugs.openjdk.java.net/browse/JDK-8030864
>>>
>>> webrev: http://cr.openjdk.java.net/~ntv/8030864/webrev.01
>>>
>>> --
>>> Thanks and Regards,
>>> Nadeesh TV
>>>
>
More information about the core-libs-dev
mailing list