fix(es/parser): emit error if await used as ident in class static ini…

[iantanwx]

…tializer

Description:

This PR fixes #8374

BREAKING CHANGE:

Related issue (if exists):

#8374

[CLAassistant]

All committers have signed the CLA.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[magic-akari]

This case should be added.

class Foo {
    static {
        function foo() {
            var await = 'bar';
        }
    }
}

[iantanwx]

This case should be added.

class Foo {
    static {
        function foo() {
            var await = 'bar';
        }
    }
}

Thanks for this, I've added it.

[kdy1]

Thanks!

---

---

swc-bump:

• swc_ecma_parser

[magic-akari]

No. It should not panic. It's valid JavaScript.

[iantanwx]

@magic-akari i didn't realise 😓

fixed it, but i'm not sure if the solution accounts for all corner cases...could you take a look?

[magic-akari]

class Foo {
    static {
        var await = "bar"; // SyntaxError: Unexpected reserved word

        function foo() {
            var await = "bar"; // ok

            class Bar {
                static {
                    var await = "bar"; // SyntaxError: Unexpected reserved word
                }
            }
        }
    }
}

I recommend setting the variable in_static_block to false upon entering the function body.
Please note that I cannot guarantee that all scenarios have been considered.

[iantanwx]

Fixed, your suggested solution seems much better to me.

[iantanwx]

@magic-akari bump