archives

« Bugzilla Issues Index

#4280 — 13.4.1 Array.prototype.toLocaleString: Unnecessary RequireObjectCoercible; Typos

• bug_id: 4280
• creation_ts: 2015-04-16 10:38:00 -0700
• short_desc: 13.4.1 Array.prototype.toLocaleString: Unnecessary RequireObjectCoercible; Typos
• delta_ts: 2015-04-16 20:44:35 -0700
• product: Internationalization - ECMA-402
• component: Specification
• version: Edition 2.0 drafts
• rep_platform: All
• op_sys: All
• bug_status: RESOLVED
• resolution: FIXED
• priority: Normal
• bug_severity: normal
• everconfirmed: true
• reporter: André Bargull
• assigned_to: Rick Waldron
• cc: waldron.rick

---

• commentid: 14192
• comment_count: 0
• who: André Bargull
• bug_when: 2015-04-16 10:38:22 -0700

13.4.1 Array.prototype.toLocaleString([locales [, options ]])

Steps 1-2 can be merged by removing RequireObjectCoercible:
> Let A be ToObject(this value)

Step 3 has a typo: "array" -> A

Steps 4-5 can be merged (Get + ToXXX calls are placed in a single step in ES2015):
> Let len be ToLength(Get(A, "length")).

Steps 12.a-b can be merged:
> Let R be ToString(Invoke(firstElement, "toLocaleString", «locales, options»)).

Steps 14.e.i-ii can be merged:
> Let R be ToString(Invoke(nextElement, "toLocaleString", «locales, options»)).

---

• commentid: 14224
• comment_count: 1
• who: Rick Waldron
• bug_when: 2015-04-16 12:47:00 -0700

13.4.1 Array.prototype.toLocaleString([locales [, options ]])

Steps 1-2 can be merged by removing RequireObjectCoercible:
> Let A be ToObject(this value)


Fixed.

> Step 3 has a typo: "array" -> A

Fixed.

> Steps 4-5 can be merged (Get + ToXXX calls are placed in a single step in ES2015):
> Let len be ToLength(Get(A, "length")).

Fixed.

> Steps 12.a-b can be merged:
> Let R be ToString(Invoke(firstElement, "toLocaleString", «locales, options»)).

Fixed.

> Steps 14.e.i-ii can be merged:
> Let R be ToString(Invoke(nextElement, "toLocaleString", «locales, options»)).

Fixed.