Closed
Bug 376564
Opened 18 years ago
Closed 18 years ago
Incorrect decompilation for "new (eval())"
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
VERIFIED
FIXED
People
(Reporter: jruderman, Assigned: jruderman)
References
Details
(Keywords: testcase)
Attachments
(1 file)
|
1.03 KB,
patch
|
brendan
:
review+
|
Details | Diff | Splinter Review |
js> function() { new (eval()) }
function () {
new eval();
}
In contrast, if I use "g" instead of "eval", the parens are kept correctly.
jsfunfuzz.js noticed this bug because it causes a round-trip change -- "new eval()" beomes "new eval" on the next trip through the decompiler.
|
Assignee | |
Comment 1•18 years ago
|
||
I'll attach a patch (similar to the patch to bug 352453). It modifies code recently added in bug 352013.
Assignee: general → jruderman
|
|
Assignee | |
Comment 2•18 years ago
|
||
|
|
Assignee | |
Updated•18 years ago
|
Attachment #260665 -
Flags: review?(brendan)
|
|
Assignee | |
Updated•18 years ago
|
Status: NEW → ASSIGNED
|
||
Comment 3•18 years ago
|
||
Comment on attachment 260665 [details] [diff] [review]
patch
Shoulda seen this one coming -- thanks!
/be
Attachment #260665 -
Flags: review?(brendan) → review+
|
|
||
Updated•18 years ago
|
|
|
Assignee | |
Comment 4•18 years ago
|
||
Patch checked in last night, 2007-04-04 20:24.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
|
||
Comment 5•18 years ago
|
||
/cvsroot/mozilla/js/tests/js1_5/decompilation/regress-376564.js,v <-- regress-376564.js
initial revision: 1.1
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•