Author: [EMAIL PROTECTED]
Date: Wed Sep 10 02:37:34 2008
New Revision: 250

Modified:
    branches/bleeding_edge/src/array.js

Log:
Avoid the creation of a string builder for joining one-element arrays.
Review URL: http://codereview.chromium.org/1888

Modified: branches/bleeding_edge/src/array.js
==============================================================================
--- branches/bleeding_edge/src/array.js (original)
+++ branches/bleeding_edge/src/array.js Wed Sep 10 02:37:34 2008
@@ -111,6 +111,14 @@
        return SparseJoin(array, length, convert);
      }

+    // Fast case for one-element arrays.
+    if (length == 1) {
+      var e = array[0];
+      if (!IS_UNDEFINED(e) || (0 in array)) {
+        return convert(e);
+      }
+    }
+
      var builder = new StringBuilder();

      for (var i = 0; i < length; i++) {

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to