Closed Bug 1841518 Opened 8 months ago Closed 7 months ago

Update Array grouping proposal implementation

Categories

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

task

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.

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
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
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.