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

chrysn's nits collected during review #109

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion draft-ietf-core-href.md
Original file line number Diff line number Diff line change
Expand Up @@ -478,7 +478,7 @@ references:
base CRI).

Application specifications that use CRIs may explicitly enable the use
of "stand-in" items (tags or simple values).
of "stand-in" items (tags or simple values other then those already in use).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what "already in use" would mean here...

Copy link
Member Author

@chrysn chrysn Feb 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What I meant was "in the CRI grammar" -- but indeed those are not stand-in items, so maybe let's just drop that change. (Like, true and false)

These are data items used in place of original representation items
such as strings or arrays, where the tag or simple value is defined to
stand for a data item that can be used in the position of the stand-in
Expand Down