The default bug view has changed. See this FAQ.

"Assertion failure: !isIndex(&dummy),"

RESOLVED FIXED in mozilla12

Status

()

Core
JavaScript Engine
--
critical
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: gkw, Assigned: Waldo)

Tracking

(Blocks: 1 bug, {assertion, regression, testcase})

Trunk
mozilla12
assertion, regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: js-triage-done)

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 586239 [details]
stack

({set"0"(l){}})

asserts js debug shell on m-c changeset 10894668e37f without any CLI arguments at Assertion failure: !isIndex(&dummy),

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   83723:d0e3133d19e2
user:        Jeff Walden
date:        Tue Dec 27 02:27:02 2011 -0600
summary:     Bug 713183 - Make JSOP_*PROP and JSOP_*NAME store a PropertyName immediate, not a JSAtom immediate, and take advantage of this fact.  r=bhackett
Created attachment 586255 [details] [diff] [review]
Patch and test

Hmm, I thought I had fixed this.  Guess not!
Assignee: general → jwalden+bmo
Status: NEW → ASSIGNED
Attachment #586255 - Flags: review?(bhackett1024)
Attachment #586255 - Flags: review?(bhackett1024) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/c2c541acf201
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: js-triage-needed → js-triage-done
Target Milestone: --- → mozilla12
Comment on attachment 586255 [details] [diff] [review]
Patch and test

>--- /dev/null
>+++ b/js/src/tests/ecma_5/Expressions/object-literal-accessor-property-name.js
>@@ -0,0 +1,30 @@
>+// Any copyright is dedicated to the Public Domain.
>+// http://creativecommons.org/licenses/publicdomain/
>+
>+var gTestfile = 'object-literal-accessor-property-name.js';
>+//-----------------------------------------------------------------------------
>+var BUGNUMBER = 536472;
>+var summary =
>+  'ES5: { get x(v) { } } and { set x(v, v2) { } } should be syntax errors';

Is that right?
Nope, not in the slightest.  Tasty, tasty copypasta with open sauce code...

https://hg.mozilla.org/integration/mozilla-inbound/rev/4bb7458176c4
https://hg.mozilla.org/mozilla-central/rev/4bb7458176c4
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Automatically extracted testcase for this bug was committed:

https://hg.mozilla.org/mozilla-central/rev/efaf8960a929
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.