Skip to content

Commit a21fd24

Browse files
fshcheglovCommit Queue
authored andcommitted
Delete deprecated TypeImpl.referencesAny and rename referencesAny2 to referencesAny.
Change-Id: I91252fbf366ddd02128315c942792ee1684e829b Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/436160 Reviewed-by: Konstantin Shcheglov <[email protected]> Commit-Queue: Paul Berry <[email protected]> Reviewed-by: Paul Berry <[email protected]>
1 parent b0838ea commit a21fd24

File tree

3 files changed

+8
-47
lines changed

3 files changed

+8
-47
lines changed

pkg/analyzer/lib/src/dart/element/least_greatest_closure.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ class LeastGreatestClosureHelper extends ReplacementVisitor {
7272
// - The greatest closure of `S` with respect to `L` is `Function`
7373
for (var typeParameter in node.typeParameters) {
7474
if (typeParameter.bound case TypeImpl bound?
75-
when bound.referencesAny2(eliminationTargets)) {
75+
when bound.referencesAny(eliminationTargets)) {
7676
return _functionReplacement;
7777
}
7878
}

pkg/analyzer/lib/src/dart/element/type.dart

Lines changed: 5 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -339,11 +339,10 @@ class FunctionTypeImpl extends TypeImpl
339339
);
340340
}
341341

342-
@Deprecated('Use referencesAny2() instead')
343342
@override
344-
bool referencesAny(Set<TypeParameterFragmentImpl> parameters) {
343+
bool referencesAny(Set<TypeParameterElementImpl> parameters) {
345344
if (typeFormals.any((element) {
346-
assert(!parameters.contains(element));
345+
assert(!parameters.contains(element.asElement2));
347346

348347
var bound = element.bound;
349348
if (bound != null && bound.referencesAny(parameters)) {
@@ -366,32 +365,6 @@ class FunctionTypeImpl extends TypeImpl
366365
return returnType.referencesAny(parameters);
367366
}
368367

369-
@override
370-
bool referencesAny2(Set<TypeParameterElementImpl> parameters) {
371-
if (typeFormals.any((element) {
372-
assert(!parameters.contains(element.asElement2));
373-
374-
var bound = element.bound;
375-
if (bound != null && bound.referencesAny2(parameters)) {
376-
return true;
377-
}
378-
379-
var defaultType = element.defaultType;
380-
return defaultType != null && defaultType.referencesAny2(parameters);
381-
})) {
382-
return true;
383-
}
384-
385-
if (this.parameters.any((element) {
386-
var type = element.type;
387-
return type.referencesAny2(parameters);
388-
})) {
389-
return true;
390-
}
391-
392-
return returnType.referencesAny2(parameters);
393-
}
394-
395368
@override
396369
TypeImpl withNullability(NullabilitySuffix nullabilitySuffix) {
397370
if (this.nullabilitySuffix == nullabilitySuffix) return this;
@@ -1130,17 +1103,11 @@ class InterfaceTypeImpl extends TypeImpl implements InterfaceType {
11301103
);
11311104
}
11321105

1133-
@Deprecated('Use referencesAny2() instead')
11341106
@override
1135-
bool referencesAny(Set<TypeParameterFragmentImpl> parameters) {
1107+
bool referencesAny(Set<TypeParameterElementImpl> parameters) {
11361108
return typeArguments.any((argument) => argument.referencesAny(parameters));
11371109
}
11381110

1139-
@override
1140-
bool referencesAny2(Set<TypeParameterElementImpl> parameters) {
1141-
return typeArguments.any((argument) => argument.referencesAny2(parameters));
1142-
}
1143-
11441111
@override
11451112
InterfaceTypeImpl withNullability(NullabilitySuffix nullabilitySuffix) {
11461113
if (this.nullabilitySuffix == nullabilitySuffix) return this;
@@ -1612,13 +1579,7 @@ abstract class TypeImpl implements DartType, SharedType {
16121579
@override
16131580
bool isStructurallyEqualTo(Object other) => this == other;
16141581

1615-
/// Returns true if this type references any of the [parameters].
1616-
@Deprecated('Use referencesAny2() instead')
1617-
bool referencesAny(Set<TypeParameterFragmentImpl> parameters) {
1618-
return false;
1619-
}
1620-
1621-
bool referencesAny2(Set<TypeParameterElementImpl> parameters) {
1582+
bool referencesAny(Set<TypeParameterElementImpl> parameters) {
16221583
return false;
16231584
}
16241585

@@ -1763,7 +1724,7 @@ class TypeParameterTypeImpl extends TypeImpl implements TypeParameterType {
17631724
}
17641725

17651726
@override
1766-
bool referencesAny2(Set<TypeParameterElementImpl> parameters) {
1727+
bool referencesAny(Set<TypeParameterElementImpl> parameters) {
17671728
return parameters.contains(element);
17681729
}
17691730

pkg/analyzer/test/src/dart/element/type_references_any_test.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,12 +60,12 @@ class TypeReferencesAnyTest extends AbstractTypeSystemTest {
6060
}
6161

6262
void _checkFalse(TypeImpl type) {
63-
var actual = type.referencesAny2({T});
63+
var actual = type.referencesAny({T});
6464
expect(actual, isFalse);
6565
}
6666

6767
void _checkTrue(TypeImpl type) {
68-
var actual = type.referencesAny2({T});
68+
var actual = type.referencesAny({T});
6969
expect(actual, isTrue);
7070
}
7171
}

0 commit comments

Comments
 (0)