Skip to content

Commit 4d36ef3

Browse files
scheglovCommit Queue
authored andcommitted
Elements. Remove MethodElementOrMember.
Change-Id: I062e1a55867ba541290ed948081cf4ea26dab2cf Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/437320 Reviewed-by: Paul Berry <[email protected]> Commit-Queue: Konstantin Shcheglov <[email protected]>
1 parent 56536ba commit 4d36ef3

File tree

3 files changed

+12
-35
lines changed

3 files changed

+12
-35
lines changed

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

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -7753,21 +7753,8 @@ class MethodElementImpl extends ExecutableElementImpl
77537753
}
77547754
}
77557755

7756-
/// Common base class for all analyzer-internal classes that implement
7757-
/// `MethodElement`.
7758-
abstract class MethodElementOrMember implements ExecutableElementOrMember {
7759-
@override
7760-
TypeImpl get returnType;
7761-
7762-
@override
7763-
FunctionTypeImpl get type;
7764-
7765-
@override
7766-
List<TypeParameterFragmentImpl> get typeParameters;
7767-
}
7768-
77697756
class MethodFragmentImpl extends ExecutableFragmentImpl
7770-
implements MethodElementOrMember, MethodFragment {
7757+
implements MethodFragment {
77717758
@override
77727759
late final MethodElementImpl element;
77737760

pkg/analyzer/lib/src/generated/error_verifier.dart

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2321,14 +2321,14 @@ class ErrorVerifier extends RecursiveAstVisitor<void>
23212321
String name = accessor.displayName;
23222322

23232323
// find inherited method or property accessor
2324-
var inherited =
2325-
_inheritanceManager
2326-
.getInherited(enclosingClass, Name(libraryUri, name))
2327-
?.asElement;
2328-
inherited ??=
2329-
_inheritanceManager
2330-
.getInherited(enclosingClass, Name(libraryUri, '$name='))
2331-
?.asElement;
2324+
var inherited = _inheritanceManager.getInherited(
2325+
enclosingClass,
2326+
Name(libraryUri, name),
2327+
);
2328+
inherited ??= _inheritanceManager.getInherited(
2329+
enclosingClass,
2330+
Name(libraryUri, '$name='),
2331+
);
23322332

23332333
if (accessor.isStatic && inherited != null) {
23342334
diagnosticReporter.atElement2(
@@ -2337,11 +2337,11 @@ class ErrorVerifier extends RecursiveAstVisitor<void>
23372337
arguments: [
23382338
enclosingClass.displayName,
23392339
name,
2340-
inherited.asElement2.enclosingElement!.displayName,
2340+
inherited.enclosingElement!.displayName,
23412341
],
23422342
);
23432343
conflictingDeclaredNames.add(name);
2344-
} else if (inherited is MethodElementOrMember) {
2344+
} else if (inherited is MethodElement2OrMember) {
23452345
// Extension type accessors preclude inherited accessors/methods.
23462346
if (enclosingClass is ExtensionTypeElementImpl) {
23472347
continue;
@@ -2352,7 +2352,7 @@ class ErrorVerifier extends RecursiveAstVisitor<void>
23522352
arguments: [
23532353
enclosingClass.displayName,
23542354
name,
2355-
inherited.asElement2.enclosingElement!.displayName,
2355+
inherited.enclosingElement!.displayName,
23562356
],
23572357
);
23582358
conflictingDeclaredNames.add(name);

pkg/analyzer/lib/src/utilities/extensions/element.dart

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -404,16 +404,6 @@ extension MethodElementImplExtension on MethodFragmentImpl {
404404
}
405405
}
406406

407-
extension MethodElementOrMemberExtension on MethodElementOrMember {
408-
MethodElement2OrMember get asElement2 {
409-
return switch (this) {
410-
MethodFragmentImpl(:var element) => element,
411-
MethodMember member => member,
412-
_ => throw UnsupportedError('Unsupported type: $runtimeType'),
413-
};
414-
}
415-
}
416-
417407
extension MixinElementImplExtension on MixinFragmentImpl {
418408
MixinElementImpl get asElement2 {
419409
return element;

0 commit comments

Comments
 (0)