New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Editorial: Rename ArrayElementList to clarify its purpose #31

Merged

gibson042 merged 2 commits into tc39:master from gibson042:2022-07-rename-arrayelementlist

Jul 17, 2022

Collaborator

gibson042 commented Jul 17, 2022 •

edited

Loading

Responding to #29 (comment)

ArrayElementList is a bit broken in general too. If elisions are present, the length of the list generated by ArrayElementList is not necessarily equal to the array's length because a single Elision can contain multiple commas but ArrayElementList treats it as a single element. However, the JSON syntax doesn't allow elisions, so it happens to work on JSON arrays.

gibson042 added 2 commits

July 17, 2022 10:59


          Editorial: Rename ArrayElementList to clarify its purpose

4cad01d


          Render spec

3ff3c7b

gibson042 merged commit 3e2cec0 into tc39:master

gibson042 mentioned this pull request

Stage 3 Specification Review: Waldemar Horwat #29

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet