Closed
Bug 817368
Opened 12 years ago
Closed 12 years ago
Map.prototype.{keys,values,entries}
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla20
People
(Reporter: jorendorff, Assigned: jorendorff)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete, Whiteboard: [DocArea=JS])
Attachments
(1 file)
13.63 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
The latest draft specifies these methods.
I'm told "items" might be renamed to "entries"; I'll get details on Monday when the latest TC39 meeting notes are posted.
Assignee | ||
Updated•12 years ago
|
Summary: Map.prototype.{keys,values,items} → Map.prototype.{keys,values,entries}
Assignee | ||
Comment 1•12 years ago
|
||
Assignee: general → jorendorff
Attachment #689856 -
Flags: review?(luke)
Comment 2•12 years ago
|
||
Comment on attachment 689856 [details] [diff] [review]
v1
Review of attachment 689856 [details] [diff] [review]:
-----------------------------------------------------------------
Nice.
::: js/src/builtin/MapObject.cpp
@@ +912,5 @@
> return js_ThrowStopIteration(cx);
> }
>
> + MapObject::IteratorKind kind = thisobj.kind();
> + if (kind == MapObject::Keys) {
Any reason not to use a switch? If you avoid 'default', you get the nice warning if you are missing an enumerator.
Attachment #689856 -
Flags: review?(luke) → review+
Updated•12 years ago
|
Keywords: dev-doc-needed
Comment 3•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Updated•11 years ago
|
Whiteboard: [DocArea=JS]
Comment 4•11 years ago
|
||
Developer release notes:
https://developer.mozilla.org/en-US/Firefox/Releases/20#JavaScript
Reference docs:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/entries
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/keys
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/values
As always, feel free to review and improve these docs in the MDN wiki. Much appreciated.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•