Closed Bug 255245 Opened 20 years ago Closed 20 years ago

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

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: brendan, Assigned: brendan)

References

()

Details

(Keywords: fixed-aviary1.0, fixed1.7, js1.5)

Attachments

(2 files)

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

Patch immediately.

/be
Attached patch fixSplinter Review
Attachment #155845 - Flags: review?(shaver)
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?
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 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+
Fixed on trunk, 1.7 branch, and aviary branch.

/be
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
thanks to be.
js1_5/Regress/regress-255245.js checked in with celsoaguiar@hotmail.com added as
contributor.
Flags: testcase+
verified fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: