Skip to content

Consider syntax of reserved-statement some more #547

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

Closed
eemeli opened this issue Dec 1, 2023 · 3 comments
Closed

Consider syntax of reserved-statement some more #547

eemeli opened this issue Dec 1, 2023 · 3 comments
Labels
Future Deferred for future standardization resolve-candidate This issue appears to have been answered or resolved, and may be closed soon. syntax Issues related with syntax or ABNF

Comments

@eemeli
Copy link
Collaborator

eemeli commented Dec 1, 2023

A few issues were raised by @gibson042 in #529 (review) that we ought to consider and address:

@stasm
Copy link
Collaborator

stasm commented Dec 1, 2023

@stasm:

What's the motiviation for putting reserved-statement inside complex-body? Do we expect other multivariant constructs than match? I think this may be building too much flexibility into the spec. Could we instead agree that any future keywords would go with other declarations?

@aphillips:

I think it was me who asked for reserved-statement to potentially be also a replacement for selectors. This would allow for a different type of selector in future syntax.

Perhaps this should be:

complex-body = quoted-pattern
             / (selectors 1*([s] variant))
              / reserved-statement ; which matches some-newly-unreserved-statement 1*([s] variant) 
                                   ; for some future statement

@aphillips aphillips added syntax Issues related with syntax or ABNF Agenda+ Requested for upcoming teleconference labels Dec 1, 2023
@aphillips
Copy link
Member

In another thread we discussed limiting reserved-statement to declarations. I have added this to the 2023-12-04 agenda.

@aphillips aphillips changed the title Consider reserved syntax some more Consider syntax of reserved-statement some more Dec 4, 2023
@aphillips aphillips added the LDML45 LDML45 Release (Tech Preview) label Jan 8, 2024
@aphillips
Copy link
Member

This was, in fact, discussed in the 2023-12-04 call and we landed the current syntax there. The status of that is that reserved-statement can appear in declarations but not as a replacement for complex-body (that is, there can only be .match or a quoted pattern in a complex message's body).

We said we would carry any additional discussion back to this issue, but no additional commentary has accrued.

I'm going to take the current syntax as consensus and mark this for resolve. If someone thinks we should allow non-.match statements in future syntax, they can file a new issue to that effect.

@aphillips aphillips added resolve-candidate This issue appears to have been answered or resolved, and may be closed soon. Future Deferred for future standardization and removed Agenda+ Requested for upcoming teleconference LDML45 LDML45 Release (Tech Preview) labels Jan 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Future Deferred for future standardization resolve-candidate This issue appears to have been answered or resolved, and may be closed soon. syntax Issues related with syntax or ABNF
Projects
None yet
Development

No branches or pull requests

3 participants