Looks good Stephen
On 18 August 2016 at 17:53, Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote: > Hello everybody! > > The factory methods of ZoneOffset class demonstrate a minor inconsistency. > Normally, invalid values of arguments are rejected (e.g. when minutes > 59), > but the value of Integer.MIN_VALUE is allowed to be passed in. > > This is due to using Math.abs(), which cannot handle Integer.MIN_VALUE well. > > Would you please help review the fix? > > BUGURL: https://bugs.openjdk.java.net/browse/JDK-8164366 > WEBREV: http://cr.openjdk.java.net/~igerasim/8164366/00/webrev/ > > With kind regards, > Ivan >
