Closed Bug 614603 Opened 9 years ago Closed 9 years ago

RegExp.length and Error.length are wrong

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: jandem, Assigned: jandem)

References

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file, 1 obsolete file)

RegExp.length is 1, should be 2. Error.length is 3, should be 1; this means undoing bug 123002, but Jeff and I think this is the right approach, because the spec explicitly requires Error.length to be 1.

This causes at least 2 sputnik failures.
Blocks: sputnik
Attached patch Patch (obsolete) — Splinter Review
Fix + updated test for Error.length and new test for RegExp.length.
Attachment #493033 - Flags: review?(jwalden+bmo)
Attached patch PatchSplinter Review
Forgot to update bug number in test; sorry for the noise.
Attachment #493033 - Attachment is obsolete: true
Attachment #493045 - Flags: review?(jwalden+bmo)
Attachment #493033 - Flags: review?(jwalden+bmo)
Attachment #493045 - Flags: review?(jwalden+bmo) → review+
http://hg.mozilla.org/mozilla-central/rev/91c193d27482
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.