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.

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

Missing UpdateExpression #524

Closed
leobalter opened this Issue Apr 6, 2016 · 0 comments

Comments

Projects
None yet
1 participant
@leobalter
Member

leobalter commented Apr 6, 2016

There's a missing part for UpdateExpression syntax at the Expressions summary (A.2 Expressions) section on the current draft:

At ES5 we can find the PostfixExpressions syntax below linking to sec 11.3:

PostfixExpression :
  LeftHandSideExpression
  LeftHandSideExpression [noLineTerminator here] ++
  LeftHandSideExpression [noLineTerminator here] --

At ES6 we found the syntax below linking to sec 12.4:

PostfixExpression[Yield] :
  LeftHandSideExpression[?Yield]
  LeftHandSideExpression[?Yield] [no LineTerminator here] ++
  LeftHandSideExpression[?Yield] [no LineTerminator here] --

At ES7 and the current draft PostfixExpression was renamed to UpdateExpression and it's found at with the following incomplete syntax:

UpdateExpression[Yield]:
  LeftHandSideExpression[?Yield][no LineTerminator here]++
  LeftHandSideExpression[?Yield][no LineTerminator here]--

The problem is apparently with the link generated from the spec.html, pointing to the syntax at the 11.9.1 Rules of Automatic Semicolon Insertion section instead of the proper 12.4 Update Expressions section.

The proper section provides the proper syntax for UpdateExpression:

UpdateExpression[Yield]:
  LeftHandSideExpression[?Yield]
  LeftHandSideExpression[?Yield][no LineTerminator here]++
  LeftHandSideExpression[?Yield][no LineTerminator here]--
  ++UnaryExpression[?Yield]
  --UnaryExpression[?Yield]

And that's what should be produced to A.2.


Apparently, the syntax at 11.9.1 being declared before 12.4 makes the process tool to use it as the valid production of UpdateExpression, fetching the wrong syntax when the Summary Annex requests it via <emu-prodref name=UpdateExpression></emu-prodref> at spec.html line 35441.

That also produces repeated ids on the generated html.

I still need to check if this bug happens on another item at the summary or using the <emu-prodref /> tag.

@bterlson bterlson closed this in bb58dff Apr 6, 2016

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