-
-
Notifications
You must be signed in to change notification settings - Fork 313
Fix accidental ABNF omission for Relative JSON Pointer #1361
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
This is good, also note |
@awwright I take it that |
|
I would propose this: <t>
The ABNF syntax of a Relative JSON Pointer is:
</t>
<sourcecode type="abnf9110"><![CDATA[
relative-json-pointer = origin-specification json-pointer
/ origin-specification "#"
; json-pointer from RFC 6901
origin-specification = non-negative-integer [ index-manipulation ]
index-manipulation = ( "+" / "-" ) non-negative-integer
non-negative-integer = "0" / %x31-39 *DIGIT
; zero, or digits without a leading zero
]]></sourcecode> In addition to the things I talked about above, this replaces the figure element with sourcecode; I changed to the |
@awwright I've incorporated your suggestion, does this look correct? |
Taking the index of the result of an index manipulation was shown in the example and intended to work, but left out of the ABNF apparently by accident. Also, rework the ABNF to keep it within the text RFC line width.
Also, use <sourcecode> tag.
The force-push / added 4 commits was just me re-basing to pick up the 2022->2023 build fix. |
Closing in favor of 👇 which has some additional commits. (☝️ was created in a fork) |
Fixes #1175 (doing another RJP draft was discussed and agreed to several months ago).
Taking the index of the result of an index manipulation was shown in the example and intended to work, but left out of the ABNF apparently by accident.
Also, rework the ABNF to keep it within the text RFC line width.