/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

Semantics of ResolveExport for default exports #601

Closed
GeorgNeis opened this Issue Jun 7, 2016 · 1 comment

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@GeorgNeis @allenwb
@GeorgNeis
Contributor

GeorgNeis commented Jun 7, 2016

If a module M does not provide a default export, then M.ResolveExport("default") throws a SyntaxError (see step 6 in 15.2.1.16.3). For any other name that cannot be found, ResolveExport returns null instead.

Throwing a SyntaxError seems wrong, also because ResolveExport is being described as side-effect free elsewhere in the spec.
@allenwb

This comment has been minimized.

Show comment
Hide comment
@allenwb

allenwb Jun 7, 2016

Member

Agreed,
Step 6.b should be Return null
Member

allenwb commented Jun 7, 2016

Agreed,
Step 6.b should be Return null

@GeorgNeis GeorgNeis referenced this issue Jun 9, 2016

Merged

ResolveExport: return null instead of throwing #606

@bterlson bterlson closed this in 0394e96 Jun 9, 2016

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