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

function parameter tracking does not handle spread syntax in arguments correctly #5502

Closed
sapphi-red opened this issue Apr 30, 2024 · 3 comments · Fixed by #5503
Closed

function parameter tracking does not handle spread syntax in arguments correctly #5502

sapphi-red opened this issue Apr 30, 2024 · 3 comments · Fixed by #5503

Comments

@sapphi-red sapphi-red changed the title function parameter tracking does not handle rest parameters correctly function parameter tracking does not handle spread syntax in arguments correctly Apr 30, 2024
@liuly0322
Copy link
Contributor

Thanks, fix in #5503. According to ecma262, The spread element can only be in the first-level of function call to change arguments length, so the fix should be enough.

Copy link

This issue has been resolved via #5503 as part of rollup@4.17.2. You can test it via npm install rollup.

@sapphi-red
Copy link
Contributor Author

Thanks!

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

Successfully merging a pull request may close this issue.

2 participants