Strict warning for trailing comma in JS object initializer

VERIFIED FIXED in M18

Status

()

P3
normal
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: taras.tielkes, Assigned: brendan)

Tracking

({js1.5})

Trunk
js1.5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
Brendan, per your request in the jseng news group:

Initializer lists that have a trailing comma shoud cause a warning in strict 
mode. (What the heck is strict mode in this context? Strict ECMAScript?).
(Reporter)

Comment 1

18 years ago
Reassigning to Brendan Eich.(at his request)
Assignee: rogerl → brendan
(Reporter)

Comment 2

18 years ago
Changed assinged-to email adress to "@mozilla.org"
Assignee: brendan → brendan
(Assignee)

Comment 3

18 years ago
Compressing Summary to fit in query pages better, adding js1.5 keyword.

/be
Status: NEW → ASSIGNED
Keywords: js1.5
Summary: Strict mode warning should be emitted for trailing comma in initializer list → Strict warning for trailing comma in JS object initializer
Target Milestone: --- → M18
(Assignee)

Comment 4

18 years ago
Created attachment 13378 [details] [diff] [review]
proposed fix, plus tab expansion and pn_op = JSOP_NEW setting for consistency
(Assignee)

Comment 5

18 years ago
Looking for r= today, this one's easy.

/be
(Assignee)

Comment 7

18 years ago
Fix checked in.

/be
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 8

18 years ago
Verifying -
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.