Closed Bug 1841518 Opened 2 years ago Closed 2 years ago

Update Array grouping proposal implementation

Categories

(Core :: JavaScript: Standard Library, task, P2)

Product:
Core
Component:
JavaScript: Standard Library
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
117 Branch
Tracking Flags:
Tracking Status
firefox117 --- fixed

People

(Reporter: anba, Assigned: anba)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

Bug 1841518: Reimplement Array grouping proposal. r=#spidermonkey-reviewers!
48 bytes, text/x-phabricator-request
Details | Review
No description provided.

Reimplement the proposal to match the new spec text.

Most important changes:

  • Methods were moved from Array.prototype to static methods on Object
    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.

Severity: -- → N/A
Priority: -- → P2
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

https://hg.mozilla.org/mozilla-central/rev/07db81a59a6b

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox117: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
Keywords: dev-doc-needed

FF117 MDN docs work for this can be tracked through https://github.com/mdn/content/issues/28279 (work done and in review)

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: