Closed Bug 559912 Opened 15 years ago Closed 14 years ago

Setter exposes JSOP_SETMETHOD optimization

Categories

(Core :: JavaScript Engine, defect)

Other Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- final+

People

(Reporter: jorendorff, Assigned: bhackett1024)

References

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file)

function s(f) { this._m = f; } function C() { Object.defineProperty(this, "m", {set: s}); this.m = function () {}; } var a = new C, b = new C; assertEq(a._m === b._m, false);
Summary: Setter defeats JSOP_SETMETHOD optimization → Setter exposes JSOP_SETMETHOD optimization
blocking2.0: --- → final+
Assignee: general → bhackett1024
Attached patch fix bogus assert (deleted) — Splinter Review
This looks to have been fixed by the changes in bug 592412 and/or bug 614051, except there is a lingering bogus assertion.
Attachment #496996 - Flags: review?(brendan)
Attachment #496996 - Flags: review?(brendan) → review+
Whiteboard: fixed-in-tracemonkey
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: