-
Notifications
You must be signed in to change notification settings - Fork 5
Changes to synchronize with N-Triples. #107
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
base: main
Are you sure you want to change the base?
Conversation
…l`, which is the correct term to reference.
spec/index.html
Outdated
@@ -1948,16 +1951,16 @@ <h3>RDF Term Constructors</h3> | |||
<tr><th> production </th><th> type </th><th>procedure</th></tr> | |||
</thead> | |||
<tbody> | |||
<tr id="handle-IRIREF" ><td style="text-align:left;" ><a class="type IRI" href="#grammar-production-IRIREF" >IRIREF </a></td><td><a data-cite="RDF12-CONCEPTS#dfn-iri" >IRI </a></td><td>The characters between <a href="#cp-less-than"><code title="less-than sign"><</code></a> and <a href="#cp-greater-than"><code title="greater-than sign">></code></a> are taken, with the <a data-lt="numeric escape sequence">numeric escape sequences</a> unescaped, to form the IRI. <a data-cite="RDF12-CONCEPTS#dfn-relative-iri">Relative IRI reference</a> resolution is performed per <a href="#sec-iri-references" class="sectionRef">Section 6.3</a>.</td></tr> | |||
<tr id="handle-IRIREF" ><td style="text-align:left;" ><a class="type IRI" href="#grammar-production-IRIREF" >IRIREF </a></td><td><a data-cite="RDF12-CONCEPTS#dfn-iri" >IRI </a></td><td>The characters between <a href="#cp-less-than"><code title="less-than sign"><</code></a> and <a href="#cp-greater-than"><code title="greater-than sign">></code></a> are taken, with the <a data-lt="numeric escape sequence">numeric escape sequences</a> unescaped, to form the IRI. <a data-cite="RDF12-CONCEPTS#dfn-relative-iri">Relative IRI reference</a> resolution is performed per <a href="#sec-iri-references" class="sectionRef">Section 6.3</a>. The resulting IRI MUST comply with the <a data-cite="RFC3987#section-2.2">syntactic restrictions</a> of generic IRI syntax, and SHOULD comply with any narrower restrictions imposed by the corresponding IRI scheme specification.</td></tr> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and SHOULD conform to section 3.3 of RFC3986
"""
The path segments "." and "..", also known as dot-segments, are
defined for relative reference within the path name hierarchy. They
are intended for use at the beginning of a relative-path reference
"""
which is advice at the level of a specific scheme except it applies to all URIs with apath.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In Turtle, do you think that wording would be repeated anywhere an IRI can be formed? (E.g., PNAME_NS/LN).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the NS is valid, can "NS:LN" be undesirable?
It might need PN_LOCAL_ESC to do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't quite get how PN_LOCAL_ESC might play into this. Either PNAME_NS or PNAME_LN can be used in the PrefixedName production, which results in an IRI that should have the same provisions as IRIREF.
We could potentially add a new "iri" term contsructor and put the requirements for the form of the resulting IRI there.
You might have a legitimate PNAME_NS IRI but when combined with PNAME_LN still not result in a valid IRI.
Co-authored-by: Andy Seaborne <[email protected]>
…ange how whitespace is introduced between `--`.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
Fixes #37. Fixes #108.
Preview | Diff