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

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

Issue 8948001: Updates dartc to recognize 'default' keyword on interface and updated factory method syntax (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Got rid of some problems. 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/FactoryNegativeTest.dart
diff --git a/tests/language/src/FactoryNegativeTest.dart b/tests/language/src/FactoryNegativeTest.dart
index d8bda6195c644c812b52e3381f9b0a4e5af2d2b4..24933220f3633f77f12913fd4775b122dbc1c8f6 100644
--- a/tests/language/src/FactoryNegativeTest.dart
+++ b/tests/language/src/FactoryNegativeTest.dart
@@ -4,19 +4,16 @@
// Test compile time error for factories with parametrized types.
-
-main() {
- // Compile time error, wrong factory method.
- var a = new Link<int>.create();
-}
-
-interface Link<T> factory LinkFactory {
+interface Link<T> factory LinkFactory { // Compile time error: should be LinkFactory<T>
ahe 2011/12/15 09:02:13 You don't have to repeat the type parameters. This
zundel 2011/12/16 21:36:29 Removed
Link.create();
}
-class LinkFactory {
- // Compile time error: should be Link<T>.create().
+class LinkFactory {
ahe 2011/12/15 09:02:13 However, this is a compile-time error.
zundel 2011/12/16 21:36:29 Added
factory Link.create() {
return null;
}
}
+
+main() {
+ var a = new Link<int>.create();
+}

Powered by Google App Engine
This is Rietveld 408576698