Assertion with Reflect.parse("default xml namespace=x")

RESOLVED FIXED

Status

()

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

People

(Reporter: Jesse Ruderman, Assigned: dherman)

Tracking

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

Trunk
x86
Mac OS X
assertion, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
Reflect.parse("default xml namespace=x")   

Assertion failure: pn->pn_arity == PN_UNARY && ((js::TokenKind)(pn->pn_u.unary.kid)->pn_type) == TOK_STRING, at ../jsreflect.cpp:1764
(Assignee)

Updated

8 years ago
Blocks: 590820
(Assignee)

Comment 1

8 years ago
Created attachment 469848 [details] [diff] [review]
default xml namespace takes an expression, not just a string literal

Fix attached.

Dave
Assignee: general → dherman
(Assignee)

Comment 2

8 years ago
Also updated the docs at

    https://developer.mozilla.org/en/SpiderMonkey/Parser_API

Dave
(Assignee)

Updated

8 years ago
Attachment #469848 - Flags: review?(cdleary)
Comment on attachment 469848 [details] [diff] [review]
default xml namespace takes an expression, not just a string literal

Straightforward.
Attachment #469848 - Flags: review?(cdleary) → review+
(Assignee)

Comment 4

8 years ago
http://hg.mozilla.org/tracemonkey/rev/690aca5702db
Whiteboard: fixed-in-tracemonkey
http://hg.mozilla.org/mozilla-central/rev/690aca5702db
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
E4X has been removed, so we won't add the test.
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.