Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Specify private reserved sigils ^ and & #404

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

Merged
merged 3 commits into from
Jul 3, 2023

Conversation

eemeli
Copy link
Collaborator

@eemeli eemeli commented Jun 30, 2023

Closes #378

This PR marks the ^ and & reserved sigils as available for implementation use, as opposed to being reserved for future spec versions. Language is also added to clarify the distinction.

reserved       = ( reserved-start / private-start ) reserved-body
reserved-start = "!" / "@" / "#" / "%" / "*" / "<" / ">" / "/" / "?" / "~"
private-start  = "^" / "&"

Of the sigils remaining for future spec versions, ! / ? and < / > seem like potential matching pairs of some sort, # could be a comment or section introducer (perhaps paired with /), and * already has a meaning as a catch-all key. That leaves the following semantically unburdened "generic" sigils: @, %, and ~. This should be sufficient for future expansion before we really need a 3.0.

@eemeli eemeli added the syntax Issues related with syntax or ABNF label Jun 30, 2023
@eemeli eemeli requested a review from aphillips June 30, 2023 03:31
Copy link
Member

@aphillips aphillips left a comment

Choose a reason for hiding this comment

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

I think we should split reserved and private-use because different things need to happen for these.

I applaud the use of the term annotation but it should be used consistently.

; and for future specification versions
reserved = ( future-start / private-start ) reserved-body
future-start = "!" / "@" / "#" / "%" / "*" / "<" / ">" / "?" / "~"
private-start = "^" / "&"
Copy link
Member

Choose a reason for hiding this comment

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

We should discuss as a group which characters to reserve for private use, although these seem as good as any.

Co-authored-by: Addison Phillips <[email protected]>
@eemeli
Copy link
Collaborator Author

eemeli commented Jul 2, 2023

Renamed future-start as reserved-start and applied other suggestions.

@eemeli eemeli requested a review from aphillips July 2, 2023 07:53
Copy link
Member

@aphillips aphillips left a comment

Choose a reason for hiding this comment

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

Thanks for the changes. This PR is in the agenda for tomorrow.

@eemeli
Copy link
Collaborator Author

eemeli commented Jul 3, 2023

I added / as a reserved-char, given that it's being considered in #398.

@aphillips aphillips merged commit caf3b0a into unicode-org:main Jul 3, 2023
@eemeli eemeli deleted the private-future branch July 3, 2023 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
syntax Issues related with syntax or ABNF
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should one or two reserved sigils be available for implementation use?
2 participants