With options('allow_xml'), the statement "" < ""; is rejected.

RESOLVED FIXED in mozilla16

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

(Blocks: 1 bug, {regression, testcase})

Trunk
mozilla16
x86_64
Mac OS X
regression, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
js> void options('allow_xml')

js> ('' < '')
false
js> '' < ''
typein:2: SyntaxError: missing ; before statement:
typein:2: '' < ''
typein:2: ...^

jsfunfuzz noticed this because the decompiler turns something that compiles into something that doesn't:

js> (function() { ('' < '') })
(function () {"" < "";})
js> (function () {"" < "";})
typein:5: SyntaxError: missing ; before statement:
typein:5: (function () {"" < "";})
typein:5: .................^

The first bad revision is:
changeset:   418ad69cfe52
user:        Benjamin Peterson
date:        Mon Jul 09 23:43:06 2012 -0700
summary:     Bug 769072: retroactively apply strict mode to defaults. r=njn

Comment 1

5 years ago
Created attachment 641370 [details] [diff] [review]
fix

Curse you, xml...
Attachment #641370 - Flags: review?(n.nethercote)
Comment on attachment 641370 [details] [diff] [review]
fix

Review of attachment 641370 [details] [diff] [review]:
-----------------------------------------------------------------

e4x cannot die quickly enough.
Attachment #641370 - Flags: review?(n.nethercote) → review+

Updated

5 years ago
Keywords: checkin-needed
Thanks for the quick fix & review!

http://hg.mozilla.org/integration/mozilla-inbound/rev/341214c1b964
Keywords: checkin-needed
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/341214c1b964
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.