Closed
Bug 420436
Opened 18 years ago
Closed 18 years ago
Fix regressions from bug 420426's patch
Categories
(Core :: JavaScript Engine, defect, P1)
Core
JavaScript Engine
Tracking
()
VERIFIED
FIXED
mozilla1.9beta4
People
(Reporter: brendan, Assigned: brendan)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
|
10.08 KB,
patch
|
Details | Diff | Splinter Review |
mrbkap suggested running the js testsuite -- how right he was. Seven regressions, more work required to make this computation fully lazy for scripted functions, without breaking fast natives, global code, with statements, and other corner cases.
/be
Flags: blocking1.9+
|
Assignee | |
Updated•18 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
|
|
Assignee | |
Comment 1•18 years ago
|
||
Attachment #306694 -
Flags: review?(mrbkap)
|
|
Assignee | |
Comment 2•18 years ago
|
||
|
||
Updated•18 years ago
|
Keywords: regression
|
||
Comment 3•18 years ago
|
||
Note, I went ahead and backed bug 420426 out before seeing this bug but think it was the right thing to do to keep the nightly clean. Shoot me if you disagree.
|
|
Assignee | |
Comment 4•18 years ago
|
||
Fixed by bc's backout; all fixes are in the latest patch in bug 420426.
/be
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
|
|
||
Comment 5•18 years ago
|
||
tests covered in bug 420426 comment 11
Flags: in-testsuite+
Flags: in-litmus-
|
||
Comment 6•18 years ago
|
||
Comment on attachment 306694 [details] [diff] [review]
fixes
This is superseded by the last patch in bug 420426.
Attachment #306694 -
Attachment is obsolete: true
Attachment #306694 -
Flags: review?(mrbkap)
You need to log in
before you can comment on or make changes to this bug.
Description
•