archives

« Bugzilla Issues Index

#4281 — 13.3.3 Date.prototype.toLocaleTimeString: Missing ReturnIfAbrupt; Unnecessary steps

• bug_id: 4281
• creation_ts: 2015-04-16 10:38:00 -0700
• short_desc: 13.3.3 Date.prototype.toLocaleTimeString: Missing ReturnIfAbrupt; Unnecessary steps
• delta_ts: 2015-04-16 20:44:31 -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: 14193
• comment_count: 0
• who: André Bargull
• bug_when: 2015-04-16 10:38:25 -0700

13.3.3 Date.prototype.toLocaleTimeString ([locales [, options ]])

Missing ReturnIfAbrupt after `thisTimeValue` in step 1.

"then" in step 2 should be elided:
> 2. If x is NaN, return "Invalid Date".

Missing ReturnIfAbrupt after ToDateTimeOptions in step 3.

Typo in step 5, replace comma with full stop.

Steps 6-8 can be merged:
> 6. Return FormatDateTime(timeFormat, x).

---

• commentid: 14221
• comment_count: 1
• who: Rick Waldron
• bug_when: 2015-04-16 12:39:52 -0700

13.3.3 Date.prototype.toLocaleTimeString ([locales [, options ]])

> Missing ReturnIfAbrupt after `thisTimeValue` in step 1.

Fixed.


"then" in step 2 should be elided:
> 2. If x is NaN, return "Invalid Date".

Fixed.


> Missing ReturnIfAbrupt after ToDateTimeOptions in step 3.

Fixed.

> Typo in step 5, replace comma with full stop.

Steps 6-8 can be merged:
> 6. Return FormatDateTime(timeFormat, x).


Fixed.