+1 on backport On Nov 27, 2014, at 5:26 PM, Hannes Wallnoefer <[email protected]> wrote:
> Unfortunately changes in Parser.java did not apply cleanly to 8u-dev. Can you > please review the webrev for 8u-dev? > > http://cr.openjdk.java.net/~hannesw/8057980/webrev-8u/ > > I also had to add a ForNode.setPerIterationScope method because of the lack > of ParserContextNodes in 8u. > > Thanks, > Hannes > > Am 2014-11-27 um 16:20 schrieb Attila Szegedi: >> +1 >> >> On Nov 27, 2014, at 1:09 PM, Hannes Wallnoefer >> <[email protected]> wrote: >> >>> Please review JDK-8057980: let & const: remaining issues with lexical >>> scoping: >>> >>> http://cr.openjdk.java.net/~hannesw/8057980/ >>> >>> Summary of changes: >>> - Throw error when let/const declaration is used in single-statement >>> context as it is not a statement >>> - Implement per-iteration scope in for(let ; ;) loop >>> - Throw error for let/const in top-level switch statement. This should be >>> supported, but we can't implement this correctly without a bigger rewrite >>> of switch statement handling. >>> - Overwriting const binding from different script should throw type error >>> also in non-strict mode. >>> - Removed unused flags: VarNode.IS_STATEMENT and ForNode.IS_FOR >>> >>> Thanks, >>> Hannes >
