Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Node constructors, EQNames (cont.). w3c/qtspecs#9 #25

Merged
merged 1 commit into from
Aug 8, 2020
Merged

Node constructors, EQNames (cont.). w3c/qtspecs#9 #25

merged 1 commit into from
Aug 8, 2020

Conversation

ChristianGruen
Copy link
Contributor

Additional tests, based on the assumption that entities in the URI string are not unescaped dynamically (see w3c/qtspecs#9 (comment)).

@michaelhkay michaelhkay merged commit 870fb5c into w3c:master Aug 8, 2020
@ChristianGruen ChristianGruen deleted the w3c/qtspecs#9 branch August 8, 2020 14:08
@michaelhkay
Copy link
Contributor

Regarding Constr-compattr-eqname-entities-1, it uses the EQName Q{ }x and assumes that this delivers a QName with namespace-uri() = " " (a single space). I'm not convinced this is correct. The spec says "The namespace URI value in a URIQualifiedName is whitespace normalized according to the rules for the xs:anyURI type in Section 3.2.17 anyURI "; it doesn't say that writing the whitespace as a character references suppresses this. Although the value space for xs:anyURI technically allows a single space, no normal method of generating an xs:anyURI value will deliver such a value; the space will always get stripped in the process of conversion from a string. Saxon, in fact, is generating a node whose namespace URI is stored as " " (a space), but this is converted to an empty string by the string-to-URI conversion performed by the namespace-uri() function. That's thoroughly unsatisfactory. I think the correct action here is for the EQName Q{ }x to represent a QName with URI="".

@ChristianGruen
Copy link
Contributor Author

You are obviously right; and I see you’ve already fixed both the attribute and element test (thanks).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants