[XL] Change in dart/sdk[main]: AST. Implement IfNullAssignment.
0 views
Skip to first unread message
Konstantin Shcheglov (Gerrit)
Aug 3, 2026, 1:25:37 AM (3 days ago) Aug 3
to Paul Berry, Johnni Winther, dart-...@luci-project-accounts.iam.gserviceaccount.com, dart-analys...@google.com, rev...@dartlang.org
Attention needed from Johnni Winther and Paul Berry
Konstantin Shcheglov voted Commit-Queue+1![]( "Open in Gerrit")
| Commit-Queue | +1 |
Related details
Attention is currently required from:
- Johnni Winther
- Paul Berry
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Paul Berry (Gerrit)
Aug 4, 2026, 12:28:52 PM (2 days ago) Aug 4
to Konstantin Shcheglov, Johnni Winther, dart-...@luci-project-accounts.iam.gserviceaccount.com, dart-analys...@google.com, rev...@dartlang.org
Attention needed from Johnni Winther and Konstantin Shcheglov
Paul Berry voted and added 2 comments![]( "Open in Gerrit")
Votes added by Paul Berry
| Code-Review | +1 |
2 comments
Patchset-level comments
File pkg/analyzer/lib/src/dart/ast/ast.dart
Line 47510, Patchset 2 (Latest):
/// A valid named read resolution.Paul Berry . unresolved
Can you expand on this comment? Other information that would be helpful includes:
- Where instances of this class are typically found
- What kinds of subtypes of this class there will be (you don't necessarily need to
enumerate them; just give a general idea of what kinds of
`ValidNamedReadResolution` there are)
Related details
Attention is currently required from:
- Johnni Winther
- Konstantin Shcheglov
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Konstantin Shcheglov (Gerrit)
Aug 5, 2026, 6:59:14 PM (9 hours ago) Aug 5
to Paul Berry, Johnni Winther, dart-...@luci-project-accounts.iam.gserviceaccount.com, dart-analys...@google.com, rev...@dartlang.org
Attention needed from Johnni Winther
Konstantin Shcheglov added 1 comment![]( "Open in Gerrit")
File pkg/analyzer/lib/src/dart/ast/ast.dart
Can you expand on this comment? Other information that would be helpful includes:
- Where instances of this class are typically found
- What kinds of subtypes of this class there will be (you don't necessarily need to
enumerate them; just give a general idea of what kinds of
`ValidNamedReadResolution` there are)
Konstantin Shcheglov
Done
Related details
Attention is currently required from:
- Johnni Winther
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Konstantin Shcheglov (Gerrit)
Aug 5, 2026, 7:13:50 PM (8 hours ago) Aug 5
to Paul Berry, Johnni Winther, dart-...@luci-project-accounts.iam.gserviceaccount.com, dart-analys...@google.com, rev...@dartlang.org
Attention needed from Johnni Winther
Konstantin Shcheglov voted Commit-Queue+2![]( "Open in Gerrit")
| Commit-Queue | +2 |
Related details
Attention is currently required from:
- Johnni Winther
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
dart-scoped@luci-project-accounts.iam.gserviceaccount.com (Gerrit)
Aug 5, 2026, 7:54:34 PM (8 hours ago) Aug 5
to Konstantin Shcheglov, Paul Berry, Johnni Winther, dart-analys...@google.com, rev...@dartlang.org
dart-...@luci-project-accounts.iam.gserviceaccount.com submitted the change with unreviewed changes![]( "Open in Gerrit")
Unreviewed changes
2 is the latest approved patch-set.
The change was submitted with unreviewed changes in the following files:
```
The name of the file: pkg/analyzer/lib/src/error/element_usage_detector.dart
Insertions: 2, Deletions: 3.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/dart/resolver/assignment_expression_resolver.dart
Insertions: 8, Deletions: 9.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/dart/ast/ast.dart
Insertions: 17, Deletions: 12.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/hint/sdk_constraint_verifier.dart
Insertions: 2, Deletions: 3.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/test/id_tests/type_promotion_test.dart
Insertions: 1, Deletions: 1.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/dart/resolver/flow_analysis_visitor.dart
Insertions: 0, Deletions: 1.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/summary2/ast_binary_writer.dart
Insertions: 2, Deletions: 4.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/generated/resolver.dart
Insertions: 6, Deletions: 5.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/test/id_tests/definite_unassignment_test.dart
Insertions: 1, Deletions: 1.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/test/id_tests/definite_assignment_test.dart
Insertions: 1, Deletions: 1.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/dart/resolver/property_element_resolver.dart
Insertions: 6, Deletions: 11.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/dart/micro/utils.dart
Insertions: 2, Deletions: 3.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/dart/analysis/index.dart
Insertions: 3, Deletions: 4.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/error/unused_local_elements_verifier.dart
Insertions: 2, Deletions: 4.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/error/best_practices_verifier.dart
Insertions: 2, Deletions: 2.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/dart/analysis/driver.dart
Insertions: 1, Deletions: 1.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/test/id_tests/nullability_test.dart
Insertions: 1, Deletions: 1.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/generated/error_verifier.dart
Insertions: 3, Deletions: 4.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/wolf/ir/ast_to_ir.dart
Insertions: 2, Deletions: 1.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/dart/analysis/search.dart
Insertions: 2, Deletions: 3.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/lib/src/summary2/ast_binary_reader.dart
Insertions: 2, Deletions: 6.
The diff is too large to show. Please review the diff.
```
```
The name of the file: pkg/analyzer/test/src/dart/resolution/assignment_test.dart
Insertions: 74, Deletions: 86.
The diff is too large to show. Please review the diff.
```
Change information
Commit message:
AST. Implement IfNullAssignment.
Change-Id: I1558556ff4b194b804d51368803f834e501a9471
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/530320
Reviewed-by: Paul Berry <paul...@google.com>
Commit-Queue: Konstantin Shcheglov <sche...@google.com>
Files:
- M pkg/analyzer/api.txt
- M pkg/analyzer/lib/analysis_rule/rule_visitor_registry.g.dart
- M pkg/analyzer/lib/dart/ast/ast.dart
- M pkg/analyzer/lib/dart/ast/visitor.g.dart
- M pkg/analyzer/lib/src/dart/analysis/driver.dart
- M pkg/analyzer/lib/src/dart/analysis/index.dart
- M pkg/analyzer/lib/src/dart/analysis/search.dart
- M pkg/analyzer/lib/src/dart/ast/ast.dart
- M pkg/analyzer/lib/src/dart/ast/ast.g.dart
- M pkg/analyzer/lib/src/dart/ast/to_source_visitor.dart
- M pkg/analyzer/lib/src/dart/micro/utils.dart
- M pkg/analyzer/lib/src/dart/resolver/assignment_expression_resolver.dart
- M pkg/analyzer/lib/src/dart/resolver/flow_analysis_visitor.dart
- M pkg/analyzer/lib/src/dart/resolver/property_element_resolver.dart
- M pkg/analyzer/lib/src/dart/resolver/this_lookup.dart
- M pkg/analyzer/lib/src/error/best_practices_verifier.dart
- M pkg/analyzer/lib/src/error/const_argument_verifier.dart
- M pkg/analyzer/lib/src/error/element_usage_detector.dart
- M pkg/analyzer/lib/src/error/unused_local_elements_verifier.dart
- M pkg/analyzer/lib/src/error/use_result_verifier.dart
- M pkg/analyzer/lib/src/fasta/ast_builder.dart
- M pkg/analyzer/lib/src/generated/error_verifier.dart
- M pkg/analyzer/lib/src/generated/resolver.dart
- M pkg/analyzer/lib/src/hint/sdk_constraint_verifier.dart
- M pkg/analyzer/lib/src/lint/linter_visitor.g.dart
- M pkg/analyzer/lib/src/summary2/ast_binary_reader.dart
- M pkg/analyzer/lib/src/summary2/ast_binary_tag.dart
- M pkg/analyzer/lib/src/summary2/ast_binary_writer.dart
- M pkg/analyzer/lib/src/summary2/informative_data.dart
- M pkg/analyzer/lib/src/test_utilities/find_node.dart
- M pkg/analyzer/lib/src/wolf/ir/ast_to_ir.dart
- M pkg/analyzer/test/generated/expression_parser_test.dart
- M pkg/analyzer/test/id_tests/definite_assignment_test.dart
- M pkg/analyzer/test/id_tests/definite_unassignment_test.dart
- M pkg/analyzer/test/id_tests/nullability_test.dart
- M pkg/analyzer/test/id_tests/type_promotion_test.dart
- M pkg/analyzer/test/src/dart/analysis/index_test.dart
- M pkg/analyzer/test/src/dart/analysis/search_test.dart
- M pkg/analyzer/test/src/dart/ast/to_source_visitor_test.dart
- M pkg/analyzer/test/src/dart/resolution/assignment_test.dart
- M pkg/analyzer/test/src/dart/resolution/non_nullable_test.dart
- M pkg/analyzer/test/src/diagnostics/unused_element_test.dart
- M pkg/analyzer/test/src/fasta/recovery/partial_code/field_declaration_test.dart
- M pkg/analyzer/test/src/fasta/recovery/partial_code/top_level_variable_test.dart
- M pkg/analyzer/test/src/summary/resolved_ast_printer.dart
- M pkg/analyzer/test/src/wolf/ir/ast_to_ir_test.dart
Change size: XL
Delta: 46 files changed, 2790 insertions(+), 449 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Paul Berry
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages