Re: RFR 8159616: java.time.Instant falls through switch statement

2016-07-01 Thread Roger Riggs
+1, looks fine. Roger On 7/1/2016 6:39 AM, Stephen Colebourne wrote: Fine by me Stephen On 1 July 2016 at 10:01, Bhanu Gopularam wrote: Hi all, May I request you to review below simple fix for following issue: Bug id: https://bugs.openjdk.java.net/browse/JDK-8159616 Solution: Removed the

Re: RFR 8159616: java.time.Instant falls through switch statement

2016-07-01 Thread Stephen Colebourne
Fine by me Stephen On 1 July 2016 at 10:01, Bhanu Gopularam wrote: > Hi all, > > May I request you to review below simple fix for following issue: > > Bug id: https://bugs.openjdk.java.net/browse/JDK-8159616 > > Solution: Removed the unneeded case statement INSTANT_SECONDS from > get(TemporalFie

RE: RFR 8159616: java.time.Instant falls through switch statement

2016-07-01 Thread Bhanu Gopularam
Hi all, May I request you to review below simple fix for following issue: Bug id: https://bugs.openjdk.java.net/browse/JDK-8159616 Solution: Removed the unneeded case statement INSTANT_SECONDS from get(TemporalField field) of java.time.Instant class Webrev: http://cr.openjdk.java.net/~bgopu

RFR 8159616: java.time.Instant falls through switch statement

2016-07-01 Thread Bhanu Gopularam
Hi all, May I request you to review below simple fix for following issue: Bug id: https://bugs.openjdk.java.net/browse/JDK-8159616 Solution: Removed the unneeded case statement INSTANT_SECONDS from get(TemporalField field) of java.time.Instant class Webrev: http://cr.openjdk.java.net/~bgopul