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

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jesse Ruderman]

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

Comment 1

[Dave Herman [:dherman]]

Attachment: default xml namespace takes an expression, not just a string literal

Fix attached.

Dave

Comment 2

[Dave Herman [:dherman]]

Also updated the docs at

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

Dave

Comment 3

[Chris Leary [:cdleary] (not checking bugmail)]

Comment on attachment 469848 [details] [diff] [review]
default xml namespace takes an expression, not just a string literal

Straightforward.

Comment 4

[Dave Herman [:dherman]]

http://hg.mozilla.org/tracemonkey/rev/690aca5702db

Comment 5

[Chris Leary [:cdleary] (not checking bugmail)]

http://hg.mozilla.org/mozilla-central/rev/690aca5702db

Comment 6

[Christian Holler (:decoder)]

E4X has been removed, so we won't add the test.