New issue
Advanced search Search tips

Issue 5486 link

Starred by 8 users

Issue metadata

Status: WontFix
Owner:
Closed: Jul 2017
Cc:
HW: ----
NextAction: ----
OS: ----
Priority: 2
Type: Bug

Blocking:
issue 1569



Sign in to add a comment

Star exports must be visited only once during ResolveExport

Project Member Reported by neis@chromium.org, Oct 6 2016

Issue description

Our ResolveExport maintains a "resolveSet" to detect cycles like the
spec does, but we forgot the "exportStarSet".
See https://tc39.github.io/ecma262/#sec-resolveexport.

The consequence is that we allow the following module A, while the
spec rejects it.

A: import {x} from "B";

B: export * from "C";

C: export var y;
   export {y as x} from "B";

I'm not sure why the spec wants to reject this, though.
 

Comment 1 by adamk@chromium.org, Oct 7 2016

Filed a spec bug to see if this can simply be removed: https://github.com/tc39/ecma262/issues/708
Labels: Priority-2

Comment 3 by neis@chromium.org, Jul 14 2017

Status: WontFix (was: Assigned)
The restriction was removed a while ago.

Sign in to add a comment