350692 - import x['y']['z']; causes "Assertion failure: pn->pn_op == JSOP_GETELEM" [@ EmitElemOp]

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jesse Ruderman]

javascript:import x['y']['z'];

Assertion failure: pn->pn_op == JSOP_GETELEM, 
at [@ EmitElemOp] jsemit.c:2343

Comment 1

[Brendan Eich [:brendan]]

Attachment: fix assertion

r=me for assertion fix.

/be

Comment 2

[Brendan Eich [:brendan]]

Fixed on trunk and 1.8 branch.

/be

Comment 3

[Bob Clary [:bc] (inactive)]

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

Comment 4

[Bob Clary [:bc] (inactive)]

verified fixed 1.8, 1.9 20060831 windows/mac*/linux

tweaked test to catch undefined x exception

Checking in regress-350692.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-350692.js,v  <--  regress-350692.js
new revision: 1.2; previous revision: 1.1

Comment 5

[Brendan Eich [:brendan]]

Or you could fake up x = {y: {z: function (){}}}; and import that, but you'd then get a TypeError: x["y"]["z"] is not exported -- could be easier to catch while avoiding catching and ignoring other exceptions that should not be ignored, in case there's a regression.

/be

Comment 6

[Bob Clary [:bc] (inactive)]

Checking in regress-350692.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-350692.js,v  <--  regress-350692.js
new revision: 1.3; previous revision: 1.2
done

Thanks for the pointer.