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

Unified Diff: tests/language/method_override2_test.dart

Issue 18600007: Relax method override restrictions. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Test and status updates. Created 7 years, 6 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
« no previous file with comments | « tests/language/language_dart2js.status ('k') | tests/language/method_override3_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/language/method_override2_test.dart
diff --git a/tests/language/method_override2_test.dart b/tests/language/method_override2_test.dart
index 12e7fc3c5bcbb30fd9ab3cdad7dfaba9dc6c261a..75fb53ad0eeb245fcf44f2f72804676325f89a95 100644
--- a/tests/language/method_override2_test.dart
+++ b/tests/language/method_override2_test.dart
@@ -5,7 +5,7 @@
// Checks that an overriding method has compatible parameters.
abstract class I {
- m({a, b});
+ m(x, {a, b});
ahe 2013/07/11 10:26:01 I don't think this change is correct. If the spec
floitsch 2013/07/11 11:18:47 Yes. I didn't have access to the latest spec and d
}
abstract class J extends I { }
@@ -15,8 +15,8 @@ abstract class K extends J {
}
class C implements I {
- m({a, b}) {
- print("$a $b");
+ m(x, {a, b}) {
+ print("$x $a $b");
}
}
@@ -33,7 +33,7 @@ class D
int main() {
var c = new C();
- c.m(a: "hello", b: "world");
+ c.m(1, a: "hello", b: "world");
var d = new D();
d.m(c: "hello", d: "world");
print("${c is I} ${d is I} ${d is I} ${d is J}");
« no previous file with comments | « tests/language/language_dart2js.status ('k') | tests/language/method_override3_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698