archives

« Bugzilla Issues Index

#4388 — Reorder arguments for PerformPromiseRace

• bug_id: 4388
• creation_ts: 2015-06-03 11:00:00 -0700
• short_desc: Reorder arguments for PerformPromiseRace
• delta_ts: 2015-06-08 16:29:15 -0700
• product: Draft for 7th Edition
• component: Deferred from 6th edition
• version: unspecified
• rep_platform: All
• op_sys: All
• bug_status: CONFIRMED
• priority: Normal
• bug_severity: enhancement
• everconfirmed: true
• reporter: C. Scott Ananian
• assigned_to: Allen Wirfs-Brock

---

• commentid: 14479
• comment_count: 0
• who: C. Scott Ananian
• bug_when: 2015-06-03 11:00:22 -0700

The arguments for PerformPromiseAll are:
( iteratorRecord, constructor, resultCapability)
The arguments for PerformPromiseRace are:
( iteratorRecord, promiseCapability, C )

It would be more pleasant if these two helpers shared the same argument order and names. I like the names for PerformPromiseAll, so that suggests that:

1. The 2nd and 3rd arguments to PerformPromiseRace be switched in the definition of Promise.race step 11.

2. The argument names be changed in the heading for 25.4.4.3.1 ("Runtime Semantics: PerformPromiseRace")

3. "promiseCapability" be changed to "resultCapability" in step 1.d.ii and 1.j in the definition of PerformPromiseRace

4. "C" be changed to "constructor" in step 1.h

If you'd like to play it safe, just switching the argument order in the heading to 25.4.4.3.1 and the callsite in Promise.race step 11 would at least help some.

---

• commentid: 14482
• comment_count: 1
• who: Allen Wirfs-Brock
• bug_when: 2015-06-08 16:29:15 -0700

deferred to es7