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

Unified Diff: frog/leg/util/link_implementation.dart

Issue 8681027: Fix bug 578 (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: prereviewed Created 9 years, 1 month 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 | « frog/lang.dart ('k') | frog/library.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: frog/leg/util/link_implementation.dart
diff --git a/frog/leg/util/link_implementation.dart b/frog/leg/util/link_implementation.dart
index 408860d432ba30c1c8e14d2f5ff773f09907ff5e..e0871b3ae3d11d74a1b1be13cd465994f6696d99 100644
--- a/frog/leg/util/link_implementation.dart
+++ b/frog/leg/util/link_implementation.dart
@@ -4,16 +4,14 @@
// TODO(ahe): This class should not be generic.
class LinkFactory {
- factory Link<T>(head, [Link tail]) {
+ factory Link<T>(T head, [Link<T> tail]) {
if (tail === null) {
- // TODO(ahe): Remove below comment about Frog bug when it is fixed.
- tail = new LinkTail<T>(); // Frog bug: T is in scope.
+ tail = new LinkTail<T>();
}
- // TODO(ahe): Remove below comment about Frog bug when it is fixed.
- return new LinkEntry<T>(head, tail); // Frog bug: T is in scope.
+ return new LinkEntry<T>(head, tail);
}
- factory Link<T>.fromList(List list) {
+ factory Link<T>.fromList(List<T> list) {
switch (list.length) {
case 0:
return new LinkTail<T>();
« no previous file with comments | « frog/lang.dart ('k') | frog/library.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698