Make argument of List.sort optional with default Comparable.compare.

lib/compiler/implementation/lib/interceptors.dart

Index: lib/compiler/implementation/lib/interceptors.dart
diff --git a/lib/compiler/implementation/lib/interceptors.dart b/lib/compiler/implementation/lib/interceptors.dart
index ffbd1d9d76b99e85cf27a00f26399fdf6debc4cf..ea48aa8b5bbcfdb838d714a04d3aa766c768870b 100644
--- a/lib/compiler/implementation/lib/interceptors.dart
+++ b/lib/compiler/implementation/lib/interceptors.dart
@@ -358,9 +358,14 @@ every(receiver, f) {
   }
 }
 
-sort(receiver, compare) {
-  if (!isJsArray(receiver)) return UNINTERCEPTED(receiver.sort(compare));
+sort$0(receiver) {
+  if (!isJsArray(receiver)) return UNINTERCEPTED(receiver.sort());
+  checkMutable(receiver, 'sort');
+  DualPivotQuicksort.sort(receiver, Comparable.compare);
+}
 
+sort$1(receiver, compare) {

[ngeoffray, 2012/10/15 14:17:12]

Why can't you have just one function with an optional parameter? Does that not
work with how we handle itnerceptors in the compiler? Is it easy to fix? Can you
add a TODO?

[Lasse Reichstein Nielsen, 2012/10/17 07:37:28]

I don't think it works with how we handle receivers (it was my first attempt,
and q.v. lastIndexOf). I have no idea how easy it is to fix. I'll add a TODO for
you.

We really should be doing interceptors differently, preferably by generating
dispatch methods from the individual specialized methods, to avoid duplicated
code (and to make method extraction work for all methods).

+  if (!isJsArray(receiver)) return UNINTERCEPTED(receiver.sort(compare));
   checkMutable(receiver, 'sort');
   DualPivotQuicksort.sort(receiver, compare);
 }