Closed
Bug 496541
Opened 15 years ago
Closed 15 years ago
TM: Different result returned with testcase involving print, for..in, return
Categories
(Core :: JavaScript Engine, defect, P2)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: gkw, Unassigned)
References
Details
(Keywords: regression, testcase)
v = 0
for each(let d in [v, v]) {
for (a in [(d--), new String]) {
print(((function(c) {
return (c)
})((function() {
return (d)
})())))
}
}
outputs
$ 28939/js-dbg-tm-intelmac -j
js> v = 0
0
js> for each(let d in [v, v]) {
for (a in [(d--), new String]) {
print(((function(c) {
return (c)
})((function() {
return (d)
})())))
}
}
-1
-1
-1
0
js>
in latest debug js shell with -j but outputs
$ 28939/js-dbg-tm-intelmac
js> v = 0
0
js> for each(let d in [v, v]) {
for (a in [(d--), new String]) {
print(((function(c) {
return (c)
})((function() {
return (d)
})())))
}
}
-1
-1
-1
-1
js>
in latest debug js shell without -j.
Flags: blocking1.9.2?
Reporter | ||
Comment 1•15 years ago
|
||
Doesn't output that last "0" with -j:
28356 - http://hg.mozilla.org/tracemonkey/rev/d62fa90d2035
Outputs that last "0" with -j:
28939 - http://hg.mozilla.org/tracemonkey/rev/6afc57314e74
No time yet to get a smaller regression window.
Keywords: regression,
regressionwindow-wanted
Reporter | ||
Comment 2•15 years ago
|
||
This does not seem to affect 1.9.1:
$ ./js-dbg-moz191-intelmac -j
js> v = 0
for each(let d in [v, v]) {
for (a in [(d--), new String]) {
print(((function(c) {
return (c)
})((function() {
return (d)
})())))
}
}
0
js> -1
-1
-1
-1
js> js>
js> ^C
$ ./js-dbg-moz191-intelmac
js> v = 0
for each(let d in [v, v]) {
for (a in [(d--), new String]) {
print(((function(c) {
return (c)
})((function() {
return (d)
})())))
}
}
0
js> -1
-1
-1
-1
js> js>
js> ^C
$
Comment 3•15 years ago
|
||
Need to know what broke this in tm/m-c and not merge it to 1.9.1.
/be
Comment 4•15 years ago
|
||
for each(let d in [0, 0]) {
d--;
for (i=0; i<4; i++)
assertEq(function(){return d;}(), -1);
}
The first bad revision is:
changeset: 28896:a16ed38ff63a
user: David Mandelin <dmandelin@mozilla.com>
date: Wed Jun 03 11:19:20 2009 -0700
summary: Bug 494269: trace JSOP_LAMBDA_FC, r=brendan,gal
Which is weird, because disfile() wants me to believe that "function(){return d;}" is using JSOP_LAMBDA, not JSOP_LAMBDA_FC.
Reporter | ||
Updated•15 years ago
|
Keywords: regressionwindow-wanted
Reporter | ||
Comment 5•15 years ago
|
||
Seems to be WFM on TM branch tm-28982-568959fa4956
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite?
Resolution: --- → WORKSFORME
Comment 6•15 years ago
|
||
Correctness bug, we should probably block on this. But, resolved fixed already.
Flags: blocking1.9.2? → blocking1.9.2+
Priority: -- → P2
You need to log in
before you can comment on or make changes to this bug.
Description
•