Closed
Bug 647423
Opened 14 years ago
Closed 9 years ago
Remove __defineGetter__ and __defineSetter__ support
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
WONTFIX
Future
People
(Reporter: bruant.d, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete, site-compat)
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.
Updated•14 years ago
|
OS: Linux → All
Hardware: x86 → All
Target Milestone: --- → Future
Version: unspecified → Trunk
Assignee | ||
Updated•10 years ago
|
Assignee: general → nobody
Comment hidden (obsolete) |
Comment 2•9 years ago
|
||
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.
Comment 3•9 years ago
|
||
jorendorff, as module owner, can you please make the call?
Flags: needinfo?(jorendorff)
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(jorendorff)
Resolution: --- → WONTFIX
Comment 4•9 years ago
|
||
This was already marked as deprecated. We recently added the "JavaScript aka Web ECMAScript" spec here.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/__defineGetter__
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/__defineSetter__
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•