archives

« Bugzilla Issues Index

#3143 — 22.1.3.24 Array.prototype.sort: Description of argument types for SortCompare

• bug_id: 3143
• creation_ts: 2014-08-20 10:34:00 -0700
• short_desc: 22.1.3.24 Array.prototype.sort: Description of argument types for SortCompare
• delta_ts: 2015-10-02 14:33:03 -0700
• product: ECMA-262, Editions 5 and 5.1
• component: technical content
• version: Edition 5.1
• rep_platform: All
• op_sys: All
• bug_status: RESOLVED
• resolution: FIXED
• see_also: https://bugs.ecmascript.org/show_bug.cgi?id=3141
• priority: Normal
• bug_severity: normal
• dependson: 3141
• everconfirmed: true
• reporter: Allen Wirfs-Brock
• assigned_to: Allen Wirfs-Brock
• cc: andrebargull

---

• commentid: 9808
• comment_count: 0
• who: Allen Wirfs-Brock
• bug_when: 2014-08-20 10:34:40 -0700

+++ This bug was initially created as a clone of Bug #3141 +++

22.1.3.24 Array.prototype.sort (comparefn)

In the "Otherwise, the following steps are taken" part:

> [...] where the arguments for calls to SortCompare are results of
> previous calls to the [[Get]] internal method.

But SortCompare takes property keys as arguments, not results from previous [[Get]] operations.

There are some recent proposals to change SortCompare to use values instead of property keys [1, 2], so basically reverting the changes from the 3rd edition. If SortCompare gets reverted to use values, this bug report can be closed for the 6th edition resp. converted to 5th edition for a future errata. IOW the mismatched description is an issue since the 3rd edition.


[1] bug 3089
[2] https://mail.mozilla.org/pipermail/es-discuss/2014-August/038847.html

---

• commentid: 9809
• comment_count: 1
• who: Allen Wirfs-Brock
• bug_when: 2014-08-20 10:36:26 -0700

This has been corrected in the 6th edition.

Marked as "In Progress" to flag as ES5.1 errata material.

---

• commentid: 14733
• comment_count: 2
• who: Brian Terlson
• bug_when: 2015-10-02 14:33:03 -0700

Bulk resolving ES5.1 errata issues as a sampling suggests these are all fixed. If this is in error, please open a new issue on GitHub.