unfork DDC's copy of most SDK libraries

sdk/lib/collection/list.dart

 // Copyright (c) 2013, 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;
 
 /**
  * Abstract implementation of a list.
  *
  * `ListBase` can be used as a base class for implementing the `List` interface.
@@ skipping 256 matching lines @@
   }
 
   void removeWhere(bool test(E element)) {
     _filter(this, test, false);
   }
 
   void retainWhere(bool test(E element)) {
     _filter(this, test, true);
   }
 
-  static void _filter(List source,
-                      bool test(var element),
-                      bool retainMatching) {
-    List retained = [];
+   void _filter(List source, bool test(var element), bool retainMatching) {

[Lasse Reichstein Nielsen, 2017/02/28 14:55:05]

Now that this is no longer a static method, the first argument can be dropped -
it's always `this`.

[Jennifer Messerly, 2017/02/28 23:46:16]

ah good catch! I made this fix over at:
https://codereview.chromium.org/2715833004/

+    List<E> retained = <E>[];
     int length = source.length;
     for (int i = 0; i < length; i++) {
       var element = source[i];
       if (test(element) == retainMatching) {
         retained.add(element);
       }
       if (length != source.length) {
         throw new ConcurrentModificationError(source);
       }
     }
@@ skipping 11 matching lines @@
     if (length == 0) {
       throw IterableElementError.noElement();
     }
     E result = this[length - 1];
     length--;
     return result;
   }
 
   void sort([int compare(E a, E b)]) {
     if (compare == null) {
-      Sort.sort(this, Comparable.compare);
+      Sort.sort(this, (a, b) => Comparable.compare(a, b));
     } else {
       Sort.sort(this, compare);
     }
   }
 
   void shuffle([Random random]) {
     if (random == null) random = new Random();
     int length = this.length;
     while (length > 1) {
       int pos = random.nextInt(length);
@@ skipping 184 matching lines @@
       for (E element in iterable) {
         this[index++] = element;
       }
     }
   }
 
   Iterable<E> get reversed => new ReversedListIterable<E>(this);
 
   String toString() => IterableBase.iterableToFullString(this, '[', ']');
 }