Re: please review patch for 8026854: java.time.temporal.TemporalQueries doesn't compile after javac modification to lambda flow analysis

2013-10-19 Thread roger riggs
Hi Vicente, Just a clarification, is the change to fully qualify the references essential or only the re-ordering to avoid the forward reference? If they are then its important to comment why; because it will not be obvious when reading the code later. Thanks, Roger On 10/18/2013 6:58 AM,

Re: please review patch for 8026854: java.time.temporal.TemporalQueries doesn't compile after javac modification to lambda flow analysis

2013-10-18 Thread Vicente-Arturo Romero-Zaldivar
On 18/10/13 13:00, Paul Sandoz wrote: On Oct 18, 2013, at 12:58 PM, Vicente-Arturo Romero-Zaldivar wrote: Hi, Please review this patch, which modifies java.time.temporal.TemporalQueries so it compiles after last changes to javac lambda flow analysis. The webrev is here: http://cr.openjdk.j

Re: please review patch for 8026854: java.time.temporal.TemporalQueries doesn't compile after javac modification to lambda flow analysis

2013-10-18 Thread Paul Sandoz
On Oct 18, 2013, at 12:58 PM, Vicente-Arturo Romero-Zaldivar wrote: > Hi, > > Please review this patch, which modifies java.time.temporal.TemporalQueries > so it compiles after last changes to javac lambda flow analysis. > > The webrev is here: > http://cr.openjdk.java.net/~vromero/8026854/w

please review patch for 8026854: java.time.temporal.TemporalQueries doesn't compile after javac modification to lambda flow analysis

2013-10-18 Thread Vicente-Arturo Romero-Zaldivar
Hi, Please review this patch, which modifies java.time.temporal.TemporalQueries so it compiles after last changes to javac lambda flow analysis. The webrev is here: http://cr.openjdk.java.net/~vromero/8026854/webrev_01/ The bug report is here: https://bugs.openjdk.java.net/browse/JDK-8026854