Can we have `AssertedPositionalParameterName` interface which contains index? about proposal-binary-ast HOT 5 CLOSED

the bug in SpiderMonkey side: https://bugzilla.mozilla.org/show_bug.cgi?id=1475458

from proposal-binary-ast.

@syg can I have your opinion, about this issue and https://github.com/binast/ecmascript-binary-ast/issues/51 and https://bugzilla.mozilla.org/show_bug.cgi?id=1475458 ?

from proposal-binary-ast.

@arai-a Sorry for missing this.

Yeah, the current parameter scope is definitely deficient for capturing positional parameters. I had hoped that isSimpleParameterList alone was sufficient -- but as your example points out, having some destructuring without parameter expressions means some arguments need to be accessed by position and some not.

The additional interface for assigning positions to asserted parameter names looks good to me.

from proposal-binary-ast.

here's WIP patch that adds above interfaces.
https://github.com/arai-a/ecmascript-binary-ast/tree/spec-draft-positional-formal
I'll open PR once rest parameter issue turns out to be solvable on SpiderMonkey side

from proposal-binary-ast.

I think we can workaround the rest parameter issue for now by a cheat (https://bugzilla.mozilla.org/show_bug.cgi?id=1475458#c9) as long as we know which is simple rest parameter.
the change in https://github.com/binast/ecmascript-binary-ast/pull/52 should help anyway.

from proposal-binary-ast.