TM: JIT affects truthiness of (false == "")

VERIFIED FIXED in mozilla1.9.1b2

Status

()

Core
JavaScript Engine
P1
critical
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: Jesse Ruderman, Assigned: gal)

Tracking

({testcase, verified1.9.1})

Trunk
mozilla1.9.1b2
x86
Mac OS X
testcase, verified1.9.1
Points:
---
Bug Flags:
blocking1.9.1 +
in-testsuite +
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
js> for (var i=0;i<5;++i) print("" + (false == ""));
true
true
false
false
false
Assignee: general → jwalden+bmo
(Reporter)

Updated

9 years ago
Blocks: 465274
(Assignee)

Comment 1

9 years ago
Created attachment 348654 [details] [diff] [review]
patch
Assignee: jwalden+bmo → gal
Attachment #348654 - Flags: review?(danderson)
Attachment #348654 - Flags: review?(danderson) → review+
(Assignee)

Comment 2

9 years ago
Fixed in TM.

http://hg.mozilla.org/tracemonkey/rev/2d07918e91eb
Severity: normal → critical
Flags: blocking1.9.1?
Priority: -- → P1
Target Milestone: --- → mozilla1.9.1b2
Fixed in the merge pushed by vlad on Nov 18 14:11:14 2008 -0800:
http://hg.mozilla.org/mozilla-central/rev/e8ed5d4bf531
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Flags: blocking1.9.1? → blocking1.9.1+

Comment 4

9 years ago
testEqFalseEmptyString already in js1_8_1/trace/trace-test.js
Flags: in-testsuite+
Flags: in-litmus-

Comment 5

9 years ago
test landed http://hg.mozilla.org/mozilla-central/rev/14b667ab591a and cvs
Keywords: fixed1.9.1

Comment 6

9 years ago
v 1.9.1, 1.9.2
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
You need to log in before you can comment on or make changes to this bug.