Index: sdk/lib/_internal/compiler/implementation/resolution/members.dart |
diff --git a/sdk/lib/_internal/compiler/implementation/resolution/members.dart b/sdk/lib/_internal/compiler/implementation/resolution/members.dart |
index 9f49b25b6012293f0d271240f46e18af5841f715..4f84f7ef5dbebbf0911ccd8a93357d7f5cb81a43 100644 |
--- a/sdk/lib/_internal/compiler/implementation/resolution/members.dart |
+++ b/sdk/lib/_internal/compiler/implementation/resolution/members.dart |
@@ -923,7 +923,6 @@ class ResolverTask extends CompilerTask { |
} |
} |
checkAbstractField(member); |
- checkValidOverride(member, cls.lookupSuperMember(member.name)); |
checkUserDefinableOperator(member); |
}); |
}); |
@@ -944,25 +943,6 @@ class ResolverTask extends CompilerTask { |
MessageKind.CONST_CONSTRUCTOR_WITH_NONFINAL_FIELDS_FIELD); |
} |
} |
- |
- if (!cls.isAbstract) { |
- for (DartType supertype in cls.allSupertypes) { |
- // This must have been reported elsewhere. |
- if (!supertype.element.isClass()) continue; |
- ClassElement superclass = supertype.element; |
- superclass.forEachMember((ClassElement holder, Element member) { |
- if (member.isAbstract) { |
- Element mine = cls.lookupMember(member.name); |
- if (mine == null || mine.isAbstract) { |
- compiler.reportWarningCode( |
- cls, MessageKind.UNIMPLEMENTED_METHOD, |
- {'class_name': cls.name, 'member_name': member.name}); |
- compiler.reportHint(member, MessageKind.THIS_IS_THE_METHOD, {}); |
- } |
- } |
- }); |
- } |
- } |
} |
void checkAbstractField(Element member) { |
@@ -1115,45 +1095,6 @@ class ResolverTask extends CompilerTask { |
compiler.reportInfo(contextElement, contextMessage); |
} |
- void checkValidOverride(Element member, Element superMember) { |
- if (superMember == null) return; |
- if (member.modifiers.isStatic()) { |
- reportErrorWithContext( |
- member, MessageKind.NO_STATIC_OVERRIDE, |
- superMember, MessageKind.NO_STATIC_OVERRIDE_CONT); |
- } else { |
- FunctionElement superFunction = superMember.asFunctionElement(); |
- FunctionElement function = member.asFunctionElement(); |
- if (superFunction == null || superFunction.isAccessor()) { |
- // Field or accessor in super. |
- if (function != null && !function.isAccessor()) { |
- // But a plain method in this class. |
- reportErrorWithContext( |
- member, MessageKind.CANNOT_OVERRIDE_FIELD_WITH_METHOD, |
- superMember, MessageKind.CANNOT_OVERRIDE_FIELD_WITH_METHOD_CONT); |
- } |
- } else { |
- // Instance method in super. |
- if (function == null || function.isAccessor()) { |
- // But a field (or accessor) in this class. |
- reportErrorWithContext( |
- member, MessageKind.CANNOT_OVERRIDE_METHOD_WITH_FIELD, |
- superMember, MessageKind.CANNOT_OVERRIDE_METHOD_WITH_FIELD_CONT); |
- } else { |
- // Both are plain instance methods. |
- if (superFunction.requiredParameterCount(compiler) != |
- function.requiredParameterCount(compiler)) { |
- reportErrorWithContext( |
- member, |
- MessageKind.BAD_ARITY_OVERRIDE, |
- superMember, |
- MessageKind.BAD_ARITY_OVERRIDE_CONT); |
- } |
- // TODO(ahe): Check optional parameters. |
- } |
- } |
- } |
- } |
FunctionSignature resolveSignature(FunctionElement element) { |
MessageKind defaultValuesError = null; |