/ecma262

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 IsFunctionDefinition for anonymous FunctionExpression #1139

Open
jorendorff opened this Issue Mar 20, 2018 · 2 comments

Comments

Assignees

@bterlson bterlson

Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@jorendorff @bakkot @anba @bterlson
@jorendorff

jorendorff commented Mar 20, 2018

See 14.1.12. It only seems to be specified for the production where the BindingIdentifier is present. Likewise, GeneratorExpression (14.4.8) and AsyncGeneratorExpression (14.5.8) only specify IsFunctionDefinition for the production with BindingIdentifier.

The corresponding spec for async functions, 14.7.8, lists both productions.

I think either the other three sections should be changed to list both productions, like 14.7.8; or else all four should be changed to use _opt.
@bakkot

This comment has been minimized.

Show comment
Hide comment
@bakkot

bakkot Mar 20, 2018

Contributor

The ? which should render as "_opt" is actually present in the source, I think (line 18841, which I can't link to since it's too big).

Pretty sure this is a bug in ecmarkup. cc @bterlson.
Contributor

bakkot commented Mar 20, 2018

The ? which should render as "_opt" is actually present in the source, I think (line 18841, which I can't link to since it's too big).

Pretty sure this is a bug in ecmarkup. cc @bterlson.

@ljharb ljharb assigned bterlson Mar 20, 2018

@anba

This comment has been minimized.

Show comment
Hide comment
@anba

anba Mar 21, 2018

Contributor

See #536
Contributor

anba commented Mar 21, 2018

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