Refactor List.setRange function.

sdk/lib/collection/collections.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // 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.
 
 part of dart.collection;
 
 /**
  * This class provides default implementations for Iterables (including Lists).
  *
  * Once Dart receives Mixins it will be replaced with mixin classes.
@@ skipping 351 matching lines @@
     if (start < 0 || start > list.length) {
       throw new RangeError.range(start, 0, list.length);
     }
     if (end < start || end > list.length) {
       throw new RangeError.range(end, start, list.length);
     }
     // The generic type is currently lost. It will be fixed with mixins.
     return new SubListIterable(list, start, end);
   }
 
-  static void setRangeList(List list, int start, int length,
+  static void setRangeList(List list, int start, int end,
                            List from, int startFrom) {
-    if (length == 0) return;
-
-    if (length < 0) throw new ArgumentError(length);
-    if (start < 0) throw new RangeError.value(start);
-    if (start + length > list.length) {
-      throw new RangeError.value(start + length);
+    if (start < 0 || start > list.length) {
+      throw new RangeError.range(start, 0, list.length);
     }
-
+    if (end < start || end > list.length) {
+      throw new RangeError.range(end, start, list.length);
+    }
+    int length = end - start;
+    if (startFrom < 0 || startFrom + length > from.length) {
+      throw new RangeError.range(startFrom, 0, from.length - length);
+    }
     Arrays.copy(from, startFrom, list, start, length);
   }
 
   static Map<int, dynamic> asMapList(List l) {
     return new ListMapView(l);
   }
 
   static bool setContainsAll(Set set, Iterable other) {
     for (var element in other) {
       if (!set.contains(element)) return false;
@@ skipping 46 matching lines @@
  * The source of the elements may be a [List] or any [Iterable] with
  * efficient [Iterable.length] and [Iterable.elementAt].
  */
 class UnmodifiableListView<E> extends UnmodifiableListBase<E> {
   Iterable<E> _source;
   /** Create an unmodifiable list backed by [source]. */
   UnmodifiableListView(Iterable<E> source) : _source = source;
   int get length => _source.length;
   E operator[](int index) => _source.elementAt(index);
 }