Skip to content

Fixes #1976. Update map pattern tests according to the new matching rules #1977

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

Merged
merged 7 commits into from
Mar 31, 2023

Conversation

sgrekhov
Copy link
Contributor

No description provided.

Copy link
Member

@eernstg eernstg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

I removed a couple of ^^^, just so the carets are aligned with the error location. (Approximately, because the location of ^^^ ignored for 'unspecified' errors, but just so we follow a consistent style).

Copy link
Member

@eernstg eernstg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

It would be really nice if we could easily see each construct which is intended to be a syntax error (the spec parser rejects many parts of this PR, and it isn't easy to see that it is all about intentional syntax errors).

I adjusted a couple of ^^^^^ lines.

@eernstg eernstg merged commit e3b5624 into dart-lang:master Mar 31, 2023
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Apr 4, 2023
2023-04-04 [email protected] Fixes dart-lang/co19#1985. Roll failures fixed (dart-lang/co19#1986)
2023-04-04 [email protected] Fixes dart-lang/co19#1983. Typo fixed in type_inference_A22_t01.dart (dart-lang/co19#1984)
2023-04-03 [email protected] Fixes dart-lang/co19#1978. Add processing of HINT.UNREACHABLE_SWITCH_CASE (dart-lang/co19#1982)
2023-04-03 [email protected] dart-lang/co19#1401. Exhaustiveness tests added (dart-lang/co19#1974)
2023-03-31 [email protected] Fixes dart-lang/co19#1976. Runtime errors fixed (dart-lang/co19#1981)
2023-03-31 [email protected] Fixes dart-lang/co19#1976. Update map pattern tests according to the new matching rules (dart-lang/co19#1977)
2023-03-31 [email protected] dart-lang/co19#1401. Additional test for the rest element of a list pattern (dart-lang/co19#1980)

Change-Id: I5be25e8f96d081028d68b5086c33b7432f1c8a66
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/292883
Commit-Queue: Erik Ernst <[email protected]>
Reviewed-by: Erik Ernst <[email protected]>
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.

2 participants