Closed
Bug 498549
Opened 15 years ago
Closed 15 years ago
TM: If the result of a demoted multiplication is 0, must undemote or we lose -0
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
status1.9.2 | --- | beta1-fixed |
People
(Reporter: gkw, Assigned: gal)
Details
(Keywords: testcase)
Attachments
(1 file)
1.64 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
e = 0 let(f = function(y) {++y; for each(b in [0, 1 / 0, 0 / 0, 0]) { e *= --y } }) { f(2) } print(uneval(this)) There's a "-0" in the output without -j but no "-0" in the output with -j. Does not seem to affect 1.9.1 branch. === $ ~/Desktop/tm-28982-568959fa4956/js-dbg-tm-intelmac js> e = 0 0 js> let(f = function(y) {++y; for each(b in [0, 1 / 0, 0 / 0, 0]) { e *= --y } }) { f(2) } js> print(uneval(this)) ({e:-0, b:0}) $ ~/Desktop/tm-28982-568959fa4956/js-dbg-tm-intelmac -j js> e = 0 0 js> let(f = function(y) {++y; for each(b in [0, 1 / 0, 0 / 0, 0]) { e *= --y } }) { f(2) } js> print(uneval(this)) ({e:0, b:0})
Flags: blocking1.9.2?
Assignee | ||
Updated•15 years ago
|
Summary: TM: Different result from testcase involving for...each, function, uneval → TM: If the result of a demoted multiplication is 0, must undemote or we lose -0
Assignee | ||
Updated•15 years ago
|
Attachment #383446 -
Flags: review?(dvander)
Updated•15 years ago
|
Attachment #383446 -
Flags: review?(dvander) → review+
Comment on attachment 383446 [details] [diff] [review] patch :(
Comment 3•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/434d3673d304
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•15 years ago
|
Flags: blocking1.9.2? → blocking1.9.2+
Reporter | ||
Updated•15 years ago
|
Flags: in-testsuite?
Comment 4•15 years ago
|
||
Mass change: adding fixed1.9.2 keyword (This bug was identified as a mozilla1.9.2 blocker which was fixed before the mozilla-1.9.2 repository was branched (August 13th, 2009) as per this query: http://is.gd/2ydcb - if this bug is not actually fixed on mozilla1.9.2, please remove the keyword. Apologies for the bugspam)
Keywords: fixed1.9.2
Updated•15 years ago
|
status1.9.2:
--- → beta1-fixed
Keywords: fixed1.9.2
Comment 5•11 years ago
|
||
Bug in removed tracer code, setting in-testsuite- flag.
Flags: in-testsuite? → in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•