Remove List.fromList constructor, and List.copyFrom. They are duplicates of the new range methods.

compiler/lib/implementation/array.dart

Index: compiler/lib/implementation/array.dart
===================================================================
--- compiler/lib/implementation/array.dart	(revision 938)
+++ compiler/lib/implementation/array.dart	(working copy)
@@ -12,15 +12,6 @@
   }
 
   factory List<E>.fromList(List<E> other, int startIndex, int endIndex) {

[floitsch, 2011/10/31 09:40:44]

forgot to remove this line.

[ngeoffray, 2011/10/31 09:47:45]

Done.

-    List list = new List<E>();
-    if (endIndex > other.length) endIndex = other.length;
-    if (startIndex < 0) startIndex = 0;
-    int count = endIndex - startIndex;
-    if (count > 0) {
-      list.length = count;
-      Arrays.copy(other, startIndex, list, 0, count);
-    }
-    return list;
   }
 
   factory List<E>([int length = null]) {
@@ -153,7 +144,10 @@
   List<T> getRange(int start, int length) {
     if (length == 0) return [];
     Arrays.rangeCheck(this, start, length);
-    return new List<T>.fromList(this, start, start + length);
+    List list = new List<T>();
+    list.length = length;
+    Arrays.copy(this, start, list, 0, length);
+    return list;
   }
 
   int indexOf(T element, int startIndex) {