-
Notifications
You must be signed in to change notification settings - Fork 23
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
Parameter binding lists should force parameters bindings to be enumerated in source order. #30
Comments
What's the problem, exactly? Default arguments? |
@efaust Is this still a problem? Do you want me to do something about this? |
CheckParameterNames checks the order in step 5: and actualParams has names in the syntactical order: so, if the order doesn't match, it's the reference encoder's bug. |
moved the issue to binjs-ref. |
Right now the reference encoder feels free to sort parameter bindings lexicographically, which doesn't play nicely with engines.
The text was updated successfully, but these errors were encountered: