Closed Bug 419662 Opened 13 years ago Closed 11 years ago
function statement and destructuring parameter name clash still favours the parameter
Another incarnation of this bug comes from bug 416737. See bug 416737 comment 6 for a test case and analysis.
Raising priority and nominating for 1.9.1 as the bug 416737, a dup of this one, is 1.9.1 P2 blocker.
Severity: minor → critical
Priority: -- → P2
The patch makes sure that the destructuring argument initializer comes as the first thing in function's bytecode.
The new patch checks for destructuring arguments only if function contains functions definitions that require the emit order change.
Attachment #370184 - Flags: review?(brendan) → review?(mrbkap)
Attachment #370184 - Flags: review?(mrbkap) → review+
The version of the patch adds missing change to the xdr version.
landed to TM - http://hg.mozilla.org/tracemonkey/rev/f6407cd260ca
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
covered by js1_7/regress/regress-410649.js and soon to be added js1_5/Regress/regress-416737-01.js, js1_5/Regress/regress-416737-02.js
You need to log in before you can comment on or make changes to this bug.