RFR: 8164669: Lazier initialization of java.time

Stephen Colebourne scolebourne at joda.org
Tue Aug 23 21:54:37 UTC 2016


Really, we should add a TemporalAmountFormatter to the JDK, but its a
bigger piece of work and quite tricky.
Stephen

On 23 August 2016 at 22:52, Claes Redestad <claes.redestad at oracle.com> wrote:
>
>
> On 2016-08-23 22:52, Stephen Colebourne wrote:
>>
>> This looks fine to me.
>
>
> Thanks for the review!
>
>> I suspect that we could hand write a parser to
>> avoid the regex, but this probably suffices.
>> Stephen
>
>
> Right, this is admittedly a bit of a hack. Maybe it would be possible
> to carefully use/extend DateTimeFormatter to deal with Durations and
> Periods?
>
> /Claes
>
>
>>
>> On 23 August 2016 at 19:49, Claes Redestad <claes.redestad at oracle.com>
>> wrote:
>>>
>>> Hi,
>>>
>>> this tiny cleanup reduces number of loaded classes from a minimal test
>>> touching java.time.ZoneId.systemDefault() by ~40, by virtue of avoiding
>>> pulling in regex and some internal Calendar-related classes.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8164669
>>> Webrev: http://cr.openjdk.java.net/~redestad/8164669/webrev.01/
>>>
>>> Thanks!
>>>
>>> /Claes


More information about the core-libs-dev mailing list