Re: [PATCH 4/4] jscript: Support nested scopes for functions defined inside.

[Jacek Caban]

Hi Paul,

On 6/11/21 7:46 PM, Paul Gofman wrote:
> Signed-off-by: Paul Gofman<pgofman(a)codeweavers.com>
> ---
>   dlls/jscript/compile.c   | 34 +++++++++++++++++++++--
>   dlls/jscript/engine.c    | 59 +++++++++++++++++++++++++++++++++++++--
>   dlls/jscript/engine.h    |  2 ++
>   dlls/jscript/parser.h    |  1 +
>   dlls/jscript/parser.y    |  1 +
>   dlls/mshtml/tests/es5.js | 60 ++++++++++++++++++++++++++++++++++++++--
>   6 files changed, 150 insertions(+), 7 deletions(-)

It would be nice to have tests for more interesting cases. I attached a 
test shows scope chain life time and it's failing for me with your patches.

While writing the test, I also noticed that 'let' inside 'for' statement 
doesn't work, for example:
for(let i = ...; ...)

Thanks,
Jacek