Closed
Bug 496422
Opened 15 years ago
Closed 15 years ago
delete of an enclosing function's argument deletes the wrong variable
Categories
(Core :: JavaScript Engine, defect, P2)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla1.9.1
People
(Reporter: mrbkap, Assigned: brendan)
References
Details
(Keywords: fixed1.9.1, Whiteboard: fixed-in-tracemonkey)
Attachments
(2 files, 1 obsolete file)
Split out from bug 496245:
js> a = 10
10
js> (function(a){ (function(){delete a;})(); })();
js> a
typein:3: ReferenceError: a is not defined
js>
Assignee | ||
Comment 1•15 years ago
|
||
Regression from ES1-3 semantics, could be more trouble than taking the patch (which is about to be attached).
/be
Assignee: general → brendan
Blocks: upvar2
Status: NEW → ASSIGNED
Flags: wanted1.9.1?
OS: Linux → All
Priority: -- → P2
Hardware: x86 → All
Target Milestone: --- → mozilla1.9.1
Assignee | ||
Comment 2•15 years ago
|
||
Attachment #382048 -
Flags: review?(igor)
Assignee | ||
Updated•15 years ago
|
Attachment #382048 -
Flags: review?(mrbkap)
Assignee | ||
Comment 3•15 years ago
|
||
Attachment #382048 -
Attachment is obsolete: true
Attachment #382051 -
Flags: review?(igor)
Attachment #382048 -
Flags: review?(mrbkap)
Attachment #382048 -
Flags: review?(igor)
Assignee | ||
Updated•15 years ago
|
Attachment #382051 -
Flags: review?(mrbkap)
Assignee | ||
Comment 4•15 years ago
|
||
The output should be
false
true
true
false
false
but the last false is true, in Firefox 3 or with this patch. Pre-existing bug in indirect eval, a WONTFIX because we will make indirect eval be global eval very soon for 1.9.2.
/be
Assignee | ||
Updated•15 years ago
|
Flags: in-testsuite?
Updated•15 years ago
|
Attachment #382051 -
Flags: review?(igor) → review+
Assignee | ||
Comment 5•15 years ago
|
||
Whiteboard: fixed-in-tracemonkey
Comment 6•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 7•15 years ago
|
||
Keywords: fixed1.9.1
Updated•15 years ago
|
Flags: wanted1.9.1? → wanted1.9.1+
Reporter | ||
Updated•15 years ago
|
Attachment #382051 -
Flags: review?(mrbkap) → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•