#1401. Execution of pattern-for statement and element. Switch expression #1870
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
eernstg
reviewed
Mar 1, 2023
There was a problem hiding this comment.
Looks good! Noted a few typos, including an initializing expression which is a list but should be a map.
One issue came up which is a bit more substantial: It looks like several object patterns have a wrong expectation (or maybe I just misremember the meaning of the instance creation expression Square(k)).
LanguageFeatures/Patterns/execution_pattern_for_element_A01_t01.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_element_A01_t01.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_element_A01_t03.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_element_A01_t05.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_element_A02_t01.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_statement_A01_t01.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_statement_A01_t01.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_statement_A01_t03.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_statement_A01_t05.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Patterns/execution_pattern_for_statement_A02_t01.dart
Outdated
Show resolved
Hide resolved
|
Tests and textts updated. Please review once again |
copybara-service bot
pushed a commit
to dart-lang/sdk
that referenced
this pull request
Mar 9, 2023
2023-03-09 [email protected] dart-lang/co19#1401. Matching tests added (dart-lang/co19#1894) 2023-03-09 [email protected] dart-lang/co19#1401. Type inference. The last part of the tests (dart-lang/co19#1859) 2023-03-09 [email protected] Fixes dart-lang/co19#1913. Update generated tests to work with class-modifiers (dart-lang/co19#1914) 2023-03-08 [email protected] Fixes dart-lang/co19#1909. Update tests not to fail with patterns feature enabled (dart-lang/co19#1910) 2023-03-08 [email protected] Fixes dart-lang/co19#1876. Don't expect type promotion in shared_case_scope_A02_t03 (dart-lang/co19#1880) 2023-03-08 [email protected] dart-lang/co19#1401. Pattern-for-in statement and element tests (dart-lang/co19#1872) 2023-03-08 [email protected] Added overloaded hashCode getter which is in accordance with == operator (dart-lang/co19#1911) 2023-03-08 [email protected] dart-lang/co19#1401. Execution of pattern-for statement and element. Switch expression (dart-lang/co19#1870) 2023-03-07 [email protected] dart-lang/co19#1401. Test for matching logical and relational patterns (dart-lang/co19#1888) 2023-03-07 [email protected] dart-lang/co19#1401. Pattern-if-case statement and element tests (dart-lang/co19#1873) 2023-03-07 [email protected] Fixes dart-lang/co19#1902. Existing tests adopted for patterns and records (dart-lang/co19#1908) 2023-03-07 [email protected] Fixes dart-lang/co19#1899. Language version 2.19 added to legacy mixins tests (dart-lang/co19#1907) 2023-03-06 [email protected] Fixes dart-lang/co19#1896. Update type inference tests failures (dart-lang/co19#1903) 2023-03-06 [email protected] Fixes dart-lang/co19#1901. Add `void` to prevent wrong failure (dart-lang/co19#1906) 2023-03-06 [email protected] Fixes dart-lang/co19#1897. Use <Object?> as an expected inferred type for List and Maps (dart-lang/co19#1905) 2023-03-06 [email protected] Fixes dart-lang/co19#1895. Fix runtime type used in the test (dart-lang/co19#1904) Change-Id: I4fc8fc07c2213f5fa05851db06bed3979d36ff52 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/287841 Commit-Queue: Jake Macdonald <[email protected]> Reviewed-by: Alexander Thomas <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pattern-for element tests here check
Listonly. Let's review it first, then I'll add tests forMapandSet