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

Unified Diff: compiler/java/com/google/dart/compiler/resolver/ResolverErrorCode.java

Issue 8632020: Make method overrides with differing parameters a compile-time error (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Report on function name, not entire function! Created 9 years 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: compiler/java/com/google/dart/compiler/resolver/ResolverErrorCode.java
diff --git a/compiler/java/com/google/dart/compiler/resolver/ResolverErrorCode.java b/compiler/java/com/google/dart/compiler/resolver/ResolverErrorCode.java
index 5e6caf23b08b25f06d076b8900ed3903ae184974..09f9c7be3657e80c4fef0d44a3d9713b82c242bb 100644
--- a/compiler/java/com/google/dart/compiler/resolver/ResolverErrorCode.java
+++ b/compiler/java/com/google/dart/compiler/resolver/ResolverErrorCode.java
@@ -24,6 +24,12 @@ public enum ResolverErrorCode implements ErrorCode {
CANNOT_INIT_STATIC_FIELD_IN_INITIALIZER("Cannot initialize a static field in an initializer list"),
CANNOT_OVERRIDE_INSTANCE_MEMBER("static member cannot override instance member %s of %s"),
CANNOT_OVERRIDE_STATIC_MEMBER("cannot override static member %s of %s"),
+ CANNOT_OVERRIDE_METHOD_WRONG_NUM_PARAMS(
+ "cannot override method %s, wrong number of parameters"),
+ CANNOT_OVERRIDE_METHOD_NUM_NAMED_PARAMS(
+ "cannot override method %s, number of named parameters doesn't match"),
+ CANNOT_OVERRIDE_METHOD_ORDER_NAMED_PARAMS(
+ "cannot override method %s, order of named parameters doesn't match"),
CANNOT_RESOLVE_CONSTRUCTOR("cannot resolve constructor %s"),
CANNOT_RESOLVE_FIELD("cannot resolve field %s"),
CANNOT_RESOLVE_LABEL("cannot resolve label %s"),

Powered by Google App Engine
This is Rietveld 408576698