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

Use input and local keywords instead of let in declarations #488

Merged
merged 5 commits into from
Oct 23, 2023

Conversation

eemeli
Copy link
Collaborator

@eemeli eemeli commented Oct 8, 2023

Implements the changes proposed in #469. Submitting this as a draft until that PR is merged as accepted.

The expression rule is here redefined as

expression = literal-expression / variable-expression / function-expression
literal-expression = "{" [s] literal [s annotation] [s] "}"
variable-expression = "{" [s] variable [s annotation] [s] "}"
function-expression = "{" [s] annotation [s] "}"

in order to make it easier for human readers, and to make variable-expression reusable in input-declaration. This isn't strictly speaking LL1, but it may be transformed to its earlier shape to make it LL1 again.

@eemeli eemeli marked this pull request as ready for review October 9, 2023 19:04
@eemeli eemeli added syntax Issues related with MF Syntax specification labels Oct 10, 2023
spec/syntax.md Outdated Show resolved Hide resolved
spec/syntax.md Outdated Show resolved Hide resolved
aphillips and others added 2 commits October 19, 2023 13:50
spec/syntax.md Outdated Show resolved Hide resolved
@eemeli eemeli merged commit 5065a51 into unicode-org:main Oct 23, 2023
1 check passed
@eemeli eemeli deleted the refactor-declarations branch October 23, 2023 19:23
@eemeli
Copy link
Collaborator Author

eemeli commented Oct 23, 2023

Merged with the modification agreed to on today's call.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
specification syntax Issues related with MF Syntax
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants