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

Disallow multiple expressions in a placeable #2

Closed
stasm opened this issue Jan 4, 2017 · 0 comments
Closed

Disallow multiple expressions in a placeable #2

stasm opened this issue Jan 4, 2017 · 0 comments
Assignees
Milestone

Comments

@stasm
Copy link
Contributor

stasm commented Jan 4, 2017

Goal

Simplify the grammar and the AST.

Description

Placeables currently allow more than one expression separated by a comma. The resulting list must be implicitly formatted with a language-specific List formatter before being interpolated into the parent pattern. Separating with a comma conflicts with the planned changes to allowing lists as selectors to select-expression.

The implicit list is also a rare construct and at the same time it impacts the shape of the AST for every placeables. I suggest to remove this feature. The { LIST(…) } syntax can be used instead. In the future we can also consider a syntax for list literals but the placeable would still have only one expression.

Discussion

https://groups.google.com/forum/#!topic/mozilla.tools.l10n/YBNCSp7J0zU

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant