Closed Bug 255245 Opened 21 years ago Closed 21 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)
Attachment #155845 - Flags: review?(shaver) → review+
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: 21 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: