Closed Bug 460504 Opened 16 years ago Closed 15 years ago

More incorrect decompilation with genexp in for-loop-condition

Categories

(Core :: JavaScript Engine, defect, P2)

Product:
Core
Component:
JavaScript Engine
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9.1b3

People

(Reporter: jruderman, Assigned: brendan)

References

Details

(Keywords: testcase) 

js> f = function () { for(; x, (1 for each (y in [])); ) { } }
function () {
    for (; x, 1 for each (y in []);) {
    }
}
js> eval(uneval(f))
typein:9: SyntaxError: missing ; after for-loop condition:
typein:9: (function () {for (; x, 1 for each (y in []);) {}})
typein:9: ..........................^

The patch in bug 443074 fixed this for "||" but not for ",".
Assignee: general → brendan
Status: NEW → ASSIGNED
OS: Mac OS X → All
Priority: -- → P2
Hardware: PC → All
Target Milestone: --- → mozilla1.9.1b3
Fixed by patch for bug 381963.

/be
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Depends on: 381963
Resolution: --- → FIXED
http://hg.mozilla.org/tracemonkey/rev/05eb7bd449e9
/cvsroot/mozilla/js/tests/js1_8/decompilation/regress-460504.js,v  <--  regress-460504.js
initial revision: 1.1
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.