350271 - "if(0x10.(a))" decomplies incorrectly

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect, P2)

Description

[Jesse Ruderman]

> function () { if(0x10.(a)) { h(); } }
function () { if (16.(a)) { h(); } }

> function () { if (16.(a)) { h(); } 
function () { if (16(a)) { h(); } }

Note that the . in the initial code appears to be the "XML filtering predicate operator" from E4X rather than property access.

The .. operator does not seem to suffer from this problem.

Comment 1

[Brendan Eich [:brendan]]

Attachment: fix

The data-driven decompiler is taking shape in my head: input precedence, output precedence, template for stack output based on arity, special quoting features...

/be

Comment 2

[Brendan Eich [:brendan]]

Fixed on trunk.

/be

Comment 3

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

Comment on attachment 235540 [details] [diff] [review]
fix

sr=shaver

Comment 4

[Mike Schroepfer]

Comment on attachment 235540 [details] [diff] [review]
fix

a=schrep/beltnzer for drivers.

Comment 5

[Brendan Eich [:brendan]]

Fixed on the 1.8 branch.

/be

Comment 6

[Bob Clary [:bc] (inactive)]

Checking in regress-350271.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-350271.js,v  <--  regress-350271.js
initial revision: 1.1

Comment 7

[Bob Clary [:bc] (inactive)]

verified fixed 1.8 20060828 windows/mac*/linux

Comment 8

[Bob Clary [:bc] (inactive)]

verified fixed 1.9 20060830 windows/mac*/linux