Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(21)

Unified Diff: editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/verifier/ErrorVerifier.java

Issue 23874024: Spec changed for INVALID_OVERRIDE_REQUIRED (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/verifier/ErrorVerifier.java
diff --git a/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/verifier/ErrorVerifier.java b/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/verifier/ErrorVerifier.java
index 1401f1e842c7e4fa7badf1389d98c549771a86eb..385fa0c3dea01d80c04c7a54f804171f5b1b60c2 100644
--- a/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/verifier/ErrorVerifier.java
+++ b/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/verifier/ErrorVerifier.java
@@ -1190,7 +1190,7 @@ public class ErrorVerifier extends RecursiveASTVisitor<Void> {
Map<String, Type> overriddenNamedPT = overriddenFT.getNamedParameterTypes();
// CTEC.INVALID_OVERRIDE_REQUIRED, CTEC.INVALID_OVERRIDE_POSITIONAL and CTEC.INVALID_OVERRIDE_NAMED
- if (overridingNormalPT.length != overriddenNormalPT.length) {
+ if (overridingNormalPT.length > overriddenNormalPT.length) {
errorReporter.reportError(
StaticWarningCode.INVALID_OVERRIDE_REQUIRED,
errorNameTarget,

Powered by Google App Engine
This is Rietveld 408576698