[jdk9] RFR: 8164366: ZoneOffset.ofHoursMinutesSeconds() does not reject invalid input

Ivan Gerasimov ivan.gerasimov at oracle.com
Fri Aug 19 12:52:36 UTC 2016


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.
>



More information about the core-libs-dev mailing list