Skip to content

Conversation

@sgrekhov
Copy link
Contributor

The previous change in this test was wrong. static M one = const MC(1); is not a potentially constant and therefore cannot be used in a constant expression.

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

@eernstg eernstg merged commit b1b6108 into dart-lang:master Apr 25, 2025
2 checks passed
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Apr 25, 2025
2025-04-25 sgrekhov22@gmail.com dart-lang/co19#3122. Expect an error in case of non-constant in a constant expression (dart-lang/co19#3159)
2025-04-24 sgrekhov22@gmail.com dart-lang/co19#3057. Add return statement in loops tests (dart-lang/co19#3158)
2025-04-24 sgrekhov22@gmail.com dart-lang/co19#3057. Update instance check tests and add negated instance check ones (dart-lang/co19#3131)
2025-04-23 sgrekhov22@gmail.com dart-lang/co19#3057. Add promotion tests for C-style for-loop (dart-lang/co19#3153)
2025-04-18 sgrekhov22@gmail.com Fixes dart-lang/co19#3122. Update constants evaluation according to the specification (dart-lang/co19#3150)

Cq-Include-Trybots: luci.dart.try:analyzer-linux-release-try
Change-Id: Ie89c9a3fbb920cb7e6488d652118ccc805b6f4ad
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/424760
Commit-Queue: Erik Ernst <eernst@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
Commit-Queue: Alexander Thomas <athom@google.com>
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