Closed Bug 352068 Opened 14 years ago Closed 14 years ago

Round-trip change with "!(3)"

Categories

(Core :: JavaScript Engine, defect, minor)

PowerPC
macOS
defect
Not set
minor

Tracking

()

VERIFIED FIXED

People

(Reporter: jruderman, Assigned: brendan)

Details

(Keywords: testcase, verified1.8.1)

Attachments

(1 file)

This is similar to bug 351693, but it involves "!" instead of the ternary operator.

./js -v 170

js> function() { return !(3) }
function () {
    return !3;
}

js> function () {
    return !3;
}
function () {
    return false;
}
Attached patch fixSplinter Review
Easy fix, safe for 1.8.1.

/be
Assignee: general → brendan
Status: NEW → ASSIGNED
Attachment #237853 - Flags: review?(mrbkap)
Attachment #237853 - Flags: approval1.8.1?
Attachment #237853 - Flags: review?(mrbkap) → review+
Comment on attachment 237853 [details] [diff] [review]
fix

a=schrep
Attachment #237853 - Flags: approval1.8.1? → approval1.8.1+
Fixed on trunk.

/be
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Fixed on the 1.8 branch.

/be
Keywords: fixed1.8.1
Checking in regress-352068.js;
/cvsroot/mozilla/js/tests/js1_7/lexical/regress-352068.js,v  <--  regress-352068.js
initial revision: 1.1
done
Flags: in-testsuite+
verified fixed 1.8 20060914 windows/linux 1.9 20060914 windows/mac*/linux
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.