355002 - "Assertion failure: 0" at jsemit.c:4414

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jesse Ruderman]

js> for each (this in []) { }
Assertion failure: 0, at jsemit.c:4414

This is a recent regression.  It also has bad effects in opt builds.

Comment 1

[Brendan Eich [:brendan]]

Attachment: fix

Sorry about that -- still tired. I need a vacation!

/be

Comment 2

[Brendan Eich [:brendan]]

I took the liberty of fixing this very recent, very bad regression on the trunk and the 1.8 branch.

/be

Comment 3

[Bob Clary [:bc] (inactive)]

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

Comment 4

[Bob Clary [:bc] (inactive)]

I don't know what I or my builds were smoking when I tested this test or checked it in. It should have been in js16 or later since it uses for each and should have caught the syntax error. I would appreciate it if someone would review and make sure it is correct.

Removing js1_5/Regress/regress-355002.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-355002.js,v  <--  regress-355002.js
new revision: delete; previous revision: 1.1
done
RCS file: /cvsroot/mozilla/js/tests/js1_6/Regress/regress-355002.js,v
done
Checking in js1_6/Regress/regress-355002.js;
/cvsroot/mozilla/js/tests/js1_6/Regress/regress-355002.js,v  <--  regress-355002.js
initial revision: 1.1
done

Comment 5

[Bob Clary [:bc] (inactive)]

verified fixed 20061003 1.8 windows/mac*/linux 1.9 windows/linux