Skip to content
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

Fix destructuring bindings #716

Open
gabejohnson opened this issue Jun 2, 2017 · 0 comments
Open

Fix destructuring bindings #716

gabejohnson opened this issue Jun 2, 2017 · 0 comments

Comments

@gabejohnson
Copy link
Member

gabejohnson commented Jun 2, 2017

#650 fixed issues w/ destructuring assignments in declarations. However, these forms

[...a, ] = b;
[, x, ...y,] = 0;
// and
[...{a=0},]=0;

should all throw with Uncaught SyntaxError: Rest element must be last element.
The issue is w/ transformDestructuring which tries to take a SpreadElement and transform it into a RestElement.

I'd like to take the tack suggested in #650 (comment)

@gabejohnson gabejohnson changed the title Fix destructured bindings Fix destructuring bindings Jun 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant