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

Unified Diff: tests/language/src/ClassOverrideNegativeTest.dart

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: tests/language/src/ClassOverrideNegativeTest.dart
diff --git a/tests/language/src/ClassOverrideNegativeTest.dart b/tests/language/src/ClassOverrideNegativeTest.dart
index 0f38760d87594a6af1b4199ab3c856701da6c175..a6e0560822755427030c8b0775b11d6e15428056 100644
--- a/tests/language/src/ClassOverrideNegativeTest.dart
+++ b/tests/language/src/ClassOverrideNegativeTest.dart
@@ -2,8 +2,8 @@
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
-// Test fails in checked mode because incompatible overriding method
-// VMOptions=--enable_type_checks
+// 8.1 Methods: It is a compile-time error if an interface method m1 overrides
+// an interface method m2 and has a different number of required parameters.
class A {
foo() {}
@@ -13,11 +13,8 @@ class B extends A {
foo(a) { }
}
-class ClassOverrideNegativeTest {
- static testMain() {
- }
-}
-
main() {
- ClassOverrideNegativeTest.testMain();
+ B instance = new B();
+ instance.foo(1);
+ print("Success");
}

Powered by Google App Engine
This is Rietveld 408576698