Skip to content

6.7.1.2. Reflective Module.ResolveExport() - Review SyntaxError on circular dependencies #55

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

Closed
caridy opened this issue Jul 14, 2015 · 2 comments

Comments

@caridy
Copy link
Contributor

caridy commented Jul 14, 2015

Reported by @bmeck (WebAssembly/design#256 (comment))

@guybedford
Copy link

I believe this refers to circular references in the bindings setup. The point is I can't do:

a.js

export {a} from './b.js';

b.js

export {a} from './a.js';

So we stop circularity of bindings, not circularity of modules.

@dherman
Copy link

dherman commented Jul 30, 2015

@guybedford is right: this part of the semantics treats re-export cycles as a static error, not cyclic module dependencies. Re-export cycles are nonsense: in the example above, a is not actually defined anywhere.

@dherman dherman closed this as completed Jul 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants