421621 - "Assertion failure: (sprop)->slot != SPROP_INVALID_SLOT || !SPROP_HAS_STUB_SETTER(sprop)" with setter, export/import

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jesse Ruderman]

var th = this;
this.__defineSetter__('x', function () {});
export *;
import th.*;
x;

Assertion failure: (sprop)->slot != SPROP_INVALID_SLOT || !SPROP_HAS_STUB_SETTER(sprop), at jsinterp.c:5022

Seems harmless in opt.

Comment 1

[Graydon Hoare :graydon]

Bug 447713 removed import/export from spidermonkey. Shall we close this bug?

Comment 2

[Brendan Eich [:brendan]]

Fixed by fix for bug 447713. Please do likewise for the others.

/be

Comment 3

[Bob Clary [:bc] (inactive)]

reopening and moving to the 1.8 branch. if we don't care, we can wontfix it.

Comment 4

[Bob Clary [:bc] (inactive)]

actually 1.8.1 and 1.9.0 are affected.

Comment 5

[Bob Clary [:bc] (inactive)]

/cvsroot/mozilla/js/tests/js1_5/extensions/regress-421621.js,v  <--  regress-421621.js
initial revision: 1.1

excluded from 1.9.1
/cvsroot/mozilla/js/tests/spidermonkey-n-1.9.1.tests,v  <--  spidermonkey-n-1.9.1.tests
new revision: 1.4; previous revision: 1.3

http://hg.mozilla.org/mozilla-central/rev/b04c04268a94

Comment 6

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Confirming that this testcase seems to work as expected in TM tip, but still occurs in 1.9.0.x branch.

Comment 7

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

(In reply to comment #2)
> Fixed by fix for bug 447713. Please do likewise for the others.
> 
> /be

Resolving Fixed is a good idea, since it works as expected now in TM tip.

Comment 8

[Bob Clary [:bc] (inactive)]

moving to trunk where this is "fixed".