Re: RFR: 8230709: Array index out of bounds in ES6 mode

2019-09-10 Thread Hannes Wallnöfer 
Am 09.09.2019 um 16:50 schrieb Attila Szegedi :
> 
> +1. Ugh. I guess our default lexical context stack is 16 deep.
> 

It is indeed :) Thanks for reviewing.

Hannes


>> On 2019. Sep 9., at 16:38, Hannes Wallnöfer  
>> wrote:
>> 
>> Please review this off-by-one array index fix that was recently reported on 
>> this list:
>> 
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8230709
>> Webrev: http://cr.openjdk.java.net/~hannesw/8230709/webrev.00/
>> 
>> Thanks,
>> Hannes
>

Re: RFR: 8230709: Array index out of bounds in ES6 mode

2019-09-09 Thread Attila Szegedi 
+1. Ugh. I guess our default lexical context stack is 16 deep.

> On 2019. Sep 9., at 16:38, Hannes Wallnöfer  
> wrote:
> 
> Please review this off-by-one array index fix that was recently reported on 
> this list:
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8230709
> Webrev: http://cr.openjdk.java.net/~hannesw/8230709/webrev.00/
> 
> Thanks,
> Hannes

RFR: 8230709: Array index out of bounds in ES6 mode

2019-09-09 Thread Hannes Wallnöfer 
Please review this off-by-one array index fix that was recently reported on 
this list:

JBS: https://bugs.openjdk.java.net/browse/JDK-8230709
Webrev: http://cr.openjdk.java.net/~hannesw/8230709/webrev.00/

Thanks,
Hannes