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 <[email protected]> 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
