Closed
Bug 839979
Opened 10 years ago
Closed 10 years ago
Implement Object.is
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla22
People
(Reporter: bruant.d, Assigned: evilpie)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(1 file, 1 obsolete file)
3.12 KB,
patch
|
jorendorff
:
review+
|
Details | Diff | Splinter Review |
Per TC39 January 2013 meeting [1], is/isnt are out as operators, but Object.is remains (no need for Object.isnt, ! is enough for negation) [1] https://github.com/rwldrn/tc39-notes/blob/master/es6/2013-01/jan-29.md#41-isisnt-operators
Assignee | ||
Comment 1•10 years ago
|
||
I implemented these as |is| and |isnt| before, taking.
Assignee: general → evilpies
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•10 years ago
|
||
I took some ideas for tests from bug 715359. The rest is pretty straight forward, except maybe the args.length() < 2, because this is actually not clearly defined. So I made a post to es-discuss.
Attachment #725793 -
Flags: review?(jorendorff)
Assignee | ||
Comment 3•10 years ago
|
||
For some reason I confused myself more than I should have here. The args.length() < 2 should not return false here. New patch coming up.
Assignee | ||
Comment 4•10 years ago
|
||
This now has the right behavior.
Attachment #725793 -
Attachment is obsolete: true
Attachment #725793 -
Flags: review?(jorendorff)
Attachment #725938 -
Flags: review?(jorendorff)
Comment 5•10 years ago
|
||
Comment on attachment 725938 [details] [diff] [review] v2 Review of attachment 725938 [details] [diff] [review]: ----------------------------------------------------------------- Beautiful!
Attachment #725938 -
Flags: review?(jorendorff) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/a321bf04c39b
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a321bf04c39b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Assignee | ||
Comment 8•10 years ago
|
||
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/is https://developer.mozilla.org/en-US/docs/Web/JavaScript/ECMAScript_6_support_in_Mozilla https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/22
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•