Remove __defineGetter__ and __defineSetter__ support

RESOLVED WONTFIX

Status

()

defect
RESOLVED WONTFIX
8 years ago
4 years ago

People

(Reporter: bruant.d, Unassigned)

Tracking

(Blocks 1 bug, {dev-doc-complete, site-compat})

Trunk
Future
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

8 years ago
In Jeff Walden's blog post (http://whereswalden.com/2010/04/16/more-spidermonkey-changes-ancient-esoteric-very-rarely-used-syntax-for-creating-getters-and-setters-is-being-removed/) is written the intention to remove __defineGetter__ and __defineSetter__ support in the longer run.

Unlike in bug 642500 (remove mutable __proto__), all use cases for __defineGetter__ and __defineSetter__ are covered by a standradized alternative; namely Object.defineProperty.
OS: Linux → All
Hardware: x86 → All
Target Milestone: --- → Future
Version: unspecified → Trunk
Assignee: general → nobody
Depends on: 1219366
I think this should be wontfixed. The only thing that happened since Waldo's blog post is that IE was, despite their claims cited in the blog post, forced to implemented them in IE11.

We don't have an infinite budget for breaking changes, so out of all the things that would actually simplify the engine, I don't think that the benefit of removing 70 lines of self-hosted JS are worth the cost.
jorendorff, as module owner, can you please make the call?
Flags: needinfo?(jorendorff)
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Flags: needinfo?(jorendorff)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.