Closed
Bug 1841518
Opened 10 months ago
Closed 10 months ago
Update Array grouping proposal implementation
Categories
(Core :: JavaScript: Standard Library, task, P2)
Core
JavaScript: Standard Library
Tracking
()
RESOLVED
FIXED
117 Branch
Tracking | Status | |
---|---|---|
firefox117 | --- | fixed |
People
(Reporter: anba, Assigned: anba)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•10 months ago
|
||
Reimplement the proposal to match the new spec text.
Most important changes:
- Methods were moved from
Array.prototype
to static methods onObject
resp.Map
. - Iteration now happens through the iteration protocol instead of using
indexed iteration.
Our implementation inlines the two abstract operations GroupBy
and
AddValueToKeyedGroup
, because that's faster and also simplifies the
implementation.
Updated•10 months ago
|
Severity: -- → N/A
Priority: -- → P2
Updated•10 months ago
|
Attachment #9342108 -
Attachment description: WIP: Bug 1841518: Reimplement Array grouping proposal. r=#spidermonkey-reviewers! → Bug 1841518: Reimplement Array grouping proposal. r=#spidermonkey-reviewers!
Pushed by andre.bargull@gmail.com: https://hg.mozilla.org/integration/autoland/rev/07db81a59a6b Reimplement Array grouping proposal. r=mgaudet
Comment 3•10 months ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
Keywords: dev-doc-needed
Comment 4•9 months ago
•
|
||
FF117 MDN docs work for this can be tracked through https://github.com/mdn/content/issues/28279 (work done and in review)
Updated•9 months ago
|
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•