Address review comments from 150935

Change-Id: Ia137c8c139e13cb9b3f8017328f5a79edfcbc0b2
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/151022
Reviewed-by: Sigmund Cherem <sigmund@google.com>
Commit-Queue: Stephen Adams <sra@google.com>
This commit is contained in:
Stephen Adams 2020-06-12 05:54:28 +00:00 committed by commit-bot@chromium.org
parent c1f0bc310d
commit bbeda37c05
3 changed files with 6 additions and 24 deletions

View file

@ -1943,10 +1943,10 @@ class TypeMap {
if (newTypesOfInterest.length > 1 ||
(newTypesOfInterest.length == 1 &&
newTypesOfInterest.single != info.declaredType)) {
// If [newTypesOfInterest] only contains the we have no information
// about the variable (it is either an instance of its declared type
// or null) and the canonical way to represent this is to have _no_
// target info.
// If [newTypesOfInterest] only contains the declared type we have no
// information about the variable (it is either an instance of its
// declared type or null) and the canonical way to represent this is
// to have _no_ target info.
TypeHolder typeHolderIfNonNull =
new TypeHolder(info.declaredType, newTypesOfInterest, null);
TypeHolder typeHolderIfNull = new TypeHolder(info.declaredType, null,

View file

@ -25,7 +25,7 @@ ComponentFactory? test(String s, [Map<String, ComponentFactory>? mf2]) {
return f;
}
work() {
main() {
Map<String, ComponentFactory> mf2 = {'int': ComponentFactory<num>()};
test('String');
@ -35,12 +35,3 @@ work() {
test('RegExp');
test('RegExp', mf2);
}
main() {
try {
work();
} catch (e, st) {
print(e);
print(st);
}
}

View file

@ -25,7 +25,7 @@ ComponentFactory test(String s, [Map<String, ComponentFactory> mf2]) {
return f;
}
work() {
main() {
Map<String, ComponentFactory> mf2 = {'int': ComponentFactory<num>()};
test('String');
@ -35,12 +35,3 @@ work() {
test('RegExp');
test('RegExp', mf2);
}
main() {
try {
work();
} catch (e, st) {
print(e);
print(st);
}
}