-
Notifications
You must be signed in to change notification settings - Fork 25
Add bad syntax tests for bnode labels with extra colons #77
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
Conversation
The productions for PN_CHARS_U are different between Turtle and N-Triples - this is visible in blank node labels. N-triples is supposed to be subset of Turtle - it appears that it is N-triples that is wrong. Turtle:
N-Triples:
This PR adds tests to |
I'll add copies to NQ and TriG. It is looking increasing certain that NT and NQ are wrong and the intention was for there not to be Let's wait to see if any parsers systems respond. |
we accept it as n-quads, as that is what the syntax says. |
The "Rio" Rust implementation passes all these tests without change (we applied the Turtle version of the rule). |
Any reason not to merge this? cc @lisp |
in itself no. |
@afs, I'd say merge at your discretion. |
See also https://www.w3.org/2001/sw/wiki/RDF1.1_Errata#erratum_30 with email discussions link. |
* Update BNF - remove colon see https://www.w3.org/2001/sw/wiki/RDF1.1_Errata#erratum_30 https://lists.w3.org/Archives/Public/public-rdf-comments/2022Mar/0000.html w3c/rdf-tests#77 https://lists.w3.org/Archives/Public/public-rdf-comments/2012Jul/0006.html --------- Co-authored-by: Gregg Kellogg <[email protected]>
* Update BNF - remove colon see https://www.w3.org/2001/sw/wiki/RDF1.1_Errata#erratum_30 https://lists.w3.org/Archives/Public/public-rdf-comments/2022Mar/0000.html w3c/rdf-tests#77 https://lists.w3.org/Archives/Public/public-rdf-comments/2012Jul/0006.html * Update HTML according BNF --------- Co-authored-by: Gregg Kellogg <[email protected]>
No description provided.