Rewrite double.parse.

runtime/lib/integers_patch.dart

Index: runtime/lib/integers_patch.dart
diff --git a/runtime/lib/integers_patch.dart b/runtime/lib/integers_patch.dart
index 05d47cf9706ab72195f810be87dd9fd8012f550a..b6ea6e134852b845d71eee72ea7f25d8a09efef2 100644
--- a/runtime/lib/integers_patch.dart
+++ b/runtime/lib/integers_patch.dart
@@ -7,26 +7,20 @@
 
 patch class int {
 
-  static bool _isWhitespace(int codePoint) {
-    return
-      (codePoint == 32) || // Space.
-      ((9 <= codePoint) && (codePoint <= 13)); // CR, LF, TAB, etc.
-  }
-
   static int _tryParseSmi(String str) {
     if (str.isEmpty) return null;
     var ix = 0;
     var endIx = str.length - 1;
     // Find first and last non-whitespace.
     while (ix <= endIx) {
-      if (!_isWhitespace(str.codeUnitAt(ix))) break;
+      if (!str._isWhitespace(str.codeUnitAt(ix))) break;

[Lasse Reichstein Nielsen, 2013/05/21 06:12:09]

This is a hack that deserves explanation. 
Anybody would expect _isWhitespace to be static/top-level here.
Is this really faster than using the same function all the time, and starting it
with a (unit <= 127) branch?

Assign str to a dynamic variable before doing this call. It should generate a
warning, since _isWhiteSpace is not on the String class.

[floitsch, 2013/05/21 14:39:31]

Has been removed.

       ix++;
     }
     if (endIx < ix) {
       return null;  // Empty.
     }
     while (endIx > ix) {
-      if (!_isWhitespace(str.codeUnitAt(endIx))) break;
+      if (!str._isWhitespace(str.codeUnitAt(endIx))) break;
       endIx--;
     }