289630 - e4x/Global/13.1.2.1.js - isXMLName (value) failures

Status: RESOLVED WONTFIX

(Core :: JavaScript Engine, defect)

Description

[Bob Clary [:bc] (inactive)]

Current trunk builds fail a number of character ranges. Update patch to test
coming up. Current results:

STATUS: 13.1.2.1 - isXMLName()
Failure messages were:
FAILED!: Section 4 of test -
FAILED!: Expected to generate exception, actual behavior: no exception was thrown
FAILED!: Section 6 of test -
FAILED!: Expected value:
FAILED!:
FAILED!: Actual value:
FAILED!: [0xAA-0xAA] : Invalid char accepted as start : Invalid Char accepted as
other
FAILED!: [0xB5-0xB5] : Invalid char accepted as start : Invalid Char accepted as
other
FAILED!: [0xB7-0xB7] : Other char not accepted
FAILED!: [0xBA-0xBA] : Invalid char accepted as start : Invalid Char accepted as
other
FAILED!: [0x132-0x133] : Invalid char accepted as start : Invalid Char accepted
as other
FAILED!: [0x13F-0x140] : Invalid char accepted as start : Invalid Char accepted
as other
FAILED!: [0x149-0x149] : Invalid char accepted as start : Invalid Char accepted
as other
FAILED!: [0x17F-0x17F] : Invalid char accepted as start : Invalid Char accepted
as other
FAILED!: [0x1C4-0x1CC] : Invalid char accepted as start : Invalid Char accepted
as other
FAILED!: [0x1F1-0x1F3] : Invalid char accepted as start : Invalid Char accepted
as other
FAILED!: [0x2B0-0x2B8] : Invalid Char accepted as other
FAILED!: [0x2BB-0x2C1] : Start char not accepted
FAILED!: [0x2E0-0x2E4] : Invalid Char accepted as other
FAILED!: [0x37A-0x37A] : Invalid Char accepted as other
FAILED!: [0x387-0x387] : Other char not accepted
FAILED!: [0x559-0x559] : Start char not accepted
FAILED!: [0x587-0x587] : Invalid char accepted as start : Invalid Char accepted
as other
FAILED!: [0x6E5-0x6E6] : Start char not accepted
FAILED!: [0xEDC-0xEDD] : Invalid char accepted as start : Invalid Char accepted
as other
FAILED!: [0x1101-0x1101] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1104-0x1104] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1108-0x1108] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x110A-0x110A] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x110D-0x110D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1113-0x113B] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x113D-0x113D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x113F-0x113F] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1141-0x114B] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x114D-0x114D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x114F-0x114F] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1151-0x1153] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1156-0x1158] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1162-0x1162] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1164-0x1164] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1166-0x1166] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1168-0x1168] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x116A-0x116C] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x116F-0x1171] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1174-0x1174] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x1176-0x119D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x119F-0x11A2] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x11A9-0x11AA] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x11AC-0x11AD] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x11B0-0x11B6] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x11B9-0x11B9] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x11BB-0x11BB] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x11C3-0x11EA] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x11EC-0x11EF] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x11F1-0x11F8] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x207F-0x207F] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x20DD-0x20E0] : Invalid Char accepted as other
FAILED!: [0x2102-0x2102] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x2107-0x2107] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x210A-0x2113] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x2115-0x2115] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x2118-0x211D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x2124-0x2124] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x2128-0x2128] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x212C-0x212D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x212F-0x2131] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x2133-0x2138] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x2160-0x217F] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0x309B-0x309C] : Invalid Char accepted as other
FAILED!: [0x3131-0x318E] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xF900-0xFA2D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB00-0xFB06] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB13-0xFB17] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB1E-0xFB1E] : Invalid Char accepted as other
FAILED!: [0xFB1F-0xFB28] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB2A-0xFB36] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB38-0xFB3C] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB3E-0xFB3E] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB40-0xFB41] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB43-0xFB44] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFB46-0xFBB1] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFBD3-0xFD3D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFD50-0xFD8F] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFD92-0xFDC7] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFDF0-0xFDFB] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFE20-0xFE23] : Invalid Char accepted as other
FAILED!: [0xFE70-0xFE72] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFE74-0xFE74] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFE76-0xFEFC] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFF10-0xFF19] : Invalid Char accepted as other
FAILED!: [0xFF21-0xFF3A] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFF41-0xFF5A] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFF66-0xFF6F] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFF70-0xFF70] : Invalid Char accepted as other
FAILED!: [0xFF71-0xFF9D] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFF9E-0xFF9F] : Invalid Char accepted as other
FAILED!: [0xFFA0-0xFFBE] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFFC2-0xFFC7] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFFCA-0xFFCF] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFFD2-0xFFD7] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!: [0xFFDA-0xFFDC] : Invalid char accepted as start : Invalid Char
accepted as other
FAILED!:
FAILED!:

Comment 1

[Bob Clary [:bc] (inactive)]

Brendan, 

I just realized my recent changes conflict with your checkin from 
1.4 brendan%mozilla.org 2005-04-05 15:47 "Don't bellow, below".

Did you want to keep the existing tests with your changes rather than the
updated ranges and test I submitted to you, Werner, Jeff and Igor?

Comment 2

[Brendan Eich [:brendan]]

Uh, the E4X spec explicitly references XML 1.0, as I tried to say in email
(maybe I was too bitter about it, and seemed to be endorsing testing for 1.1
names).  We should test the current spec, not some future revision to track XML 1.1.

/be

Comment 3

[Bob Clary [:bc] (inactive)]

I based my ranges on http://www.w3.org/TR/REC-xml/#IDAIO1S which is in
"Extensible Markup Language (XML) 1.0 (Third Edition)" which is the same
document you referenced in your email. I assumed from "Painful to fix, but a
one-time change." that you meant for the test to match that production. Whatever.

Comment 4

[Brendan Eich [:brendan]]

The test I see at top of trunk does not match the character ranges at
http://www.w3.org/TR/REC-xml/#NT-BaseChar -- in particular, the test does

pushInterval(buffer, 0xF8, 0x2FF);

but the spec has

[#x00F8-#x00FF] | [#x0100-#x0131] | [#x0134-#x013E] | [#x0141-#x0148] |
[#x014A-#x017E] | [#x0180-#x01C3] | [#x01CD-#x01F0] | [#x01F4-#x01F5] |
[#x01FA-#x0217] | [#x0250-#x02A8] | [#x02BB-#x02C1]

/be

Comment 5

[Bob Clary [:bc] (inactive)]

The ranges I used are in:
http://bclary.com/2004/10/03/js-tests/e4x/Global/13.1.2.1.js

Comment 6

[Brendan Eich [:brendan]]

D'oh!  Sorry about that.

I wonder whether this is due to SpiderMonkey using Unicode 2 era tables? :-/

/be

Comment 7

[Bob Clary [:bc] (inactive)]

(In reply to comment #6)

> I wonder whether this is due to SpiderMonkey using Unicode 2 era tables? :-/

Don't know much about Unicode unfortunately. Should I commit my new version of
the test or try to get the current version to match? I like the new version
since it seems clearer in how it is written and in interpreting the ranges of
failures. Your call.

Comment 8

[Bob Clary [:bc] (inactive)]

I went ahead and checked in the new version of the test.

Checking in 13.1.2.1.js;
/cvsroot/mozilla/js/tests/e4x/Global/13.1.2.1.js,v  <--  13.1.2.1.js
new revision: 1.5; previous revision: 1.4

Comment 9

[Tom S [:evilpie]]

With my new unicode code, I could fix this.

Comment 10

[Matthias Versen [:Matti]]

E4X will be removed again from Spidermonkey (bug 788293)