proposal-binary-ast
proposal-binary-ast copied to clipboard
Parameter binding lists should force parameters bindings to be enumerated in source order.
Right now the reference encoder feels free to sort parameter bindings lexicographically, which doesn't play nicely with engines.
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: https://binast.github.io/ecmascript-binary-ast/#sec-checkparameternames
and actualParams has names in the syntactical order: https://tc39.github.io/ecma262/#sec-function-definitions-static-semantics-boundnames
so, if the order doesn't match, it's the reference encoder's bug. I'll check it shortly
moved the issue to binjs-ref. I'd suggest closing this issue.