287909 - Rhino parser does not accept E4X comment literals

Status: RESOLVED INACTIVE

(Rhino Graveyard :: E4X, defect)

Description

[Martin Honnen]

The section 11.1.4 XML Initialiser in the E4X specification says:

"An XML initialiser is an expression describing the initialization of an XML
object, written in a form of a literal. It may specify an XML element, an XML
comment, an XML PI, or a CDATA section using ordinary XML syntax."

That suggests to me that it should possible to write an XML initializer
representing a single comment for instance. And indeed the test case

XML.ignoreComments = false;
print("XML.ignoreComments: " + XML.ignoreComments);
var comment = <!-- Kibology for all. -->;
if (comment) {
  print('comment.nodeKind(): ' + comment.nodeKind());
}
else {
  print('comment === undefined: ' + (comment === undefined));
}

when run in Spidermonkey is parsed and executed without problems:

js> load('mozillaBugs/e4x/singleComment1.js');
XML.ignoreComments: false
comment.nodeKind(): comment

With Rhino however parsing the test case yields syntax error messages, no
execution happens:

Rhino 1.6 release 1 2004 11 30
js> load('mozillaBugs/e4x/singleComment1.js');
js: "C:\Dokumente und Einstellungen\All
Users\Dokumente\javascript\mozillaBugs\e4x\singleComment1.js", line 4: syntax error
js: if (comment) {
js: ..^
js: "C:\Dokumente und Einstellungen\All
Users\Dokumente\javascript\mozillaBugs\e4x\singleComment1.js", line 6: syntax error
js: }
js: ^
js: "C:\Dokumente und Einstellungen\All
Users\Dokumente\javascript\mozillaBugs\e4x\singleComment1.js", line 1:
Compilation produced 2 syntax errors.

If the Spidermonkey behaviour is correct then this is a bug in Rhino.

Comment 1

[Martin Honnen]

Attachment: test case to be run in Rhino or Spidermonkey shell

Comment 2

[Igor Bukanov]

Reassigning to [e-mail] pending resolution of bug 288433

Comment 3

[Sylvestre Ledru [:Sylvestre]]

Closing. Bug management is now done here:
https://github.com/mozilla/rhino