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

Skip to content

Sort the reserved sigils, check for completeness #587

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.

Sign up for GitHub

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

Jump to bottom
Closed
aphillips opened this issue Jan 10, 2024 · 2 comments · Fixed by #592
Closed

Sort the reserved sigils, check for completeness #587

aphillips opened this issue Jan 10, 2024 · 2 comments · Fixed by #592
Labels
editorial Issue is non-normative LDML45 LDML45 Release (Tech Preview) syntax Issues related with syntax or ABNF

Comments

@aphillips
Copy link
Member

In #574 I suggested that the sigils should be sorted in the ABNF. This is to track that comment.

currently we have:

reserved-annotation-start = "!" / "@" / "%" / "*" / "+"
                          / "<" / ">" / "?" / "~"

I would suggest this, which is in code point order:

reserved-annotation-start = "!" / "%" / "*" / "+"
                          / "<" / ">" / "?" / "@" / "~"

Also, are we missing these ASCII punctuation items: :, ;, (, ) ? (Note that & and ^ are private use sigils)
@aphillips aphillips added syntax Issues related with syntax or ABNF editorial Issue is non-normative LDML45 LDML45 Release (Tech Preview) labels Jan 10, 2024
@eemeli
Copy link
Collaborator

eemeli commented Jan 12, 2024

We can't reserve : because we're using it for :functions, and ; is too close to it in appearance.

If we do want to consider (), then we should probably also consider []. But that's best done in a separate issue.

@eemeli eemeli mentioned this issue Jan 12, 2024
Merged
@aphillips
Copy link
Member Author

Sorry, not : for sure. I don't think semi-colon would be wrong to reserve. We just probably never want to use it. I didn't put [/] because we had farted around with using those for keys... but that ship apparently has sailed. Since this is addressed by one of your PRs, we can make another issue for enclosing reserved characters (/) and [/].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
editorial Issue is non-normative LDML45 LDML45 Release (Tech Preview) syntax Issues related with syntax or ABNF
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add @attributes as reserved unicode-org/message-format-wg
2 participants
@aphillips @eemeli