Thanks Nadeesh. It's a good catch!
Here's the updated webrev:
http://cr.openjdk.java.net/~igerasim/8164366/01/webrev/
I also slightly modified comareTo(), not because there was an error in
it, but just to avoid thinking too much about possible overflow in
subtraction (of course, there can be no overflow here, as totalSeconds
is bounded.)
Now we just need official blessing from the Reviewer.
With kind regards,
Ivan
On 19.08.2016 9:01, nadeesh tv wrote:
Hi Ivan,
ZoneOffset.ofTotalSeconds(Integer.MIN_VALUE) have also the same issue.
It' not throwing the expected DTE.
May be you can correct this also as part of this.
Please update the copyright year also.
Rest looks good.
