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

Clarify the evaluation order for collection literal elements #2175

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

eernstg
Copy link
Member

@eernstg eernstg commented Mar 30, 2022

The evaluation order for the elements of a collection literal (say, [e1, ...e2, if (b) e3] where e2 is a set) was not mentioned. This PR adds a few words to clarify that evaluation occurs in textual order. The order of evaluation associated with the subsequences with other than a single element is already specified (e.g., the subsequence for a Set spread is obtained by a pseudo-code for-in statement that explicitly states how each object is obtained).

It also corrects a couple of typos where an empty sequence followed by a semicolon was shown as [[;]] rather than [[]];. One more set of typos fixed: \ell changed to \ell_1, four locations, in spec of static analysis of <forElement>.

The wording around "object sequences" is not wonderful, but I'd recommend that we leave it as is for now.

@eernstg eernstg requested a review from lrhn March 30, 2022 09:19
@github-actions
Copy link

github-actions bot commented Mar 30, 2022

Visit the preview URL for this PR (updated for commit 28f889e):

https://dart-specification--pr2175-specify-collection-l-7u4mbzdr.web.app

(expires Wed, 06 Apr 2022 15:46:34 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6941ecd630c4f067ff3d02708a45ae0f0a42b88a

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

Successfully merging this pull request may close these issues.

1 participant