Function.prototype.toSource/.toString show "setrval" instead of "return"

VERIFIED FIXED

Status

()

VERIFIED FIXED
14 years ago
12 years ago

People

(Reporter: brendan, Assigned: brendan)

Tracking

({fixed-aviary1.0, fixed1.7, js1.5})

Trunk
fixed-aviary1.0, fixed1.7, js1.5
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Assignee)

Description

14 years ago
function f() {
    try {
    } catch (e) {
        return false;
    }
    finally {
    }
}
print(f);

Patch immediately.

/be
(Assignee)

Updated

14 years ago
Attachment #155845 - Flags: review?(shaver)
Attachment #155845 - Flags: review?(shaver) → review+

Comment 2

14 years ago
This got to be the fastest bug fix on a module I've ever seen. It was literally,
send a message to jseng, go to the bathroom, return to the office and see the
patch ready! A great THANK YOU Brendan. Do you guys also provide tickets to haven?
(Assignee)

Comment 3

14 years ago
Comment on attachment 155845 [details] [diff] [review]
fix

Easy fix, good for all active branches.

/be
Attachment #155845 - Flags: approval1.8a3?
Attachment #155845 - Flags: approval1.7.3?
Attachment #155845 - Flags: approval-aviary?

Comment 4

14 years ago
Comment on attachment 155845 [details] [diff] [review]
fix

a=asa for checkin to managed branches.
Attachment #155845 - Flags: approval1.8a3?
Attachment #155845 - Flags: approval1.8a3+
Attachment #155845 - Flags: approval1.7.3?
Attachment #155845 - Flags: approval1.7.3+
Attachment #155845 - Flags: approval-aviary?
Attachment #155845 - Flags: approval-aviary+
(Assignee)

Comment 5

14 years ago
Fixed on trunk, 1.7 branch, and aviary branch.

/be
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Keywords: fixed-aviary1.0, fixed1.7
Resolution: --- → FIXED

Comment 6

14 years ago
Created attachment 174923 [details]
js1_5/Regress/regress-255245.js

thanks to be.

Comment 7

14 years ago
js1_5/Regress/regress-255245.js checked in with celsoaguiar@hotmail.com added as
contributor.

Updated

14 years ago
Flags: testcase+

Comment 8

12 years ago
verified fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.