-
Notifications
You must be signed in to change notification settings - Fork 28
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
Fixes #2304. Add more Object
member tests
#2312
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Just one little twist could be useful: It would be good if we could verify in all cases that the tools use the most specific (combined) member signature for all Object
members, rather than unconditionally using the signature from Object
.
LanguageFeatures/Extension-types/static_analysis_member_invocation_A01_t04.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Extension-types/static_analysis_member_invocation_A01_t03.dart
Show resolved
Hide resolved
LanguageFeatures/Extension-types/static_analysis_member_invocation_A01_t03.dart
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated. Please take another look
LanguageFeatures/Extension-types/static_analysis_member_invocation_A01_t03.dart
Show resolved
Hide resolved
LanguageFeatures/Extension-types/static_analysis_member_invocation_A01_t04.dart
Outdated
Show resolved
Hide resolved
LanguageFeatures/Extension-types/static_analysis_member_invocation_A01_t03.dart
Show resolved
Hide resolved
TypeET t2 = e2.runtimeType; | ||
|
||
bool b1 = e1 == e1; | ||
BoolET b2 = e2 == e2; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool!
LanguageFeatures/Extension-types/static_analysis_member_invocation_A01_t03.dart
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
2023-10-23 sgrekhov22@gmail.com Fixes dart-lang/co19#2319. Fix roll failures (dart-lang/co19#2321) 2023-10-23 sgrekhov22@gmail.com dart-lang/co19#1400. [Extension types] Add more top types tests (dart-lang/co19#2322) 2023-10-18 sgrekhov22@gmail.com dart-lang/co19#1400. [Extension types] Add more superinterfaces tests (dart-lang/co19#2315) 2023-10-18 sgrekhov22@gmail.com dart-lang/co19#2291. Add more Link.createSync() tests. Part 2 (dart-lang/co19#2316) 2023-10-17 sgrekhov22@gmail.com Fixes dart-lang/co19#2304. Add more `Object` member tests (dart-lang/co19#2312) 2023-10-17 sgrekhov22@gmail.com dart-lang/co19#2313. Fix expected static type in upper_bound_A01_t05.dart (dart-lang/co19#2314) Change-Id: Id5279b7bad5e45c7e8a5d2fa7cbffe49bd1b2093 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/331860 Reviewed-by: Erik Ernst <eernst@google.com> Reviewed-by: Alexander Thomas <athom@google.com>
No description provided.