Skip to content

replace invalid JS according to newest spec#679

Merged
arai-a merged 1 commit into
mozilla-spidermonkey:masterfrom
Archkon:remov-invalid-js
Jul 15, 2026
Merged

replace invalid JS according to newest spec#679
arai-a merged 1 commit into
mozilla-spidermonkey:masterfrom
Archkon:remov-invalid-js

Conversation

@Archkon

@Archkon Archkon commented Sep 27, 2025

Copy link
Copy Markdown
Contributor

This demo code would cause SyntaxError: The left-hand side of a for-of loop may not be 'async' among main js runtime

refer to https://tc39.es/ecma262/#sec-for-in-and-for-of-statements

@arai-a arai-a left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for working on this.

Unfortunately this change doesn't fit the adjacent context.

Comment thread js-quirks.md
@Archkon
Archkon marked this pull request as draft September 29, 2025 16:19
@Archkon
Archkon force-pushed the remov-invalid-js branch from f768c6b to a366364 Compare July 12, 2026 13:08
@Archkon Archkon changed the title remove invalid JS according to newest spec replace invalid JS according to newest spec Jul 12, 2026
@Archkon
Archkon marked this pull request as ready for review July 12, 2026 13:12
which will cause SyntaxError: The left-hand side of a for-of loop may not be 'async'

refer to https://tc39.es/ecma262/#sec-for-in-and-for-of-statements
@Archkon
Archkon force-pushed the remov-invalid-js branch from a366364 to f43f508 Compare July 12, 2026 13:16
@Archkon
Archkon requested a review from arai-a July 12, 2026 13:17

@arai-a arai-a left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@arai-a
arai-a merged commit e14ee38 into mozilla-spidermonkey:master Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants