Flatten the Arrays returned and consumed by the v8::Map API

src/collection.js

Index: src/collection.js
diff --git a/src/collection.js b/src/collection.js
index 1c6b475df6cf4761f4d25738150b1d3bfa3c44d5..9dd5bfb7f923cb4d2169af4f8cb96f1afff43d07 100644
--- a/src/collection.js
+++ b/src/collection.js
@@ -485,9 +485,10 @@ $getExistingHash = GetExistingHash;
 $mapFromArray = function(array) {
   var map = new GlobalMap;
   var length = array.length;
-  for (var i = 0; i < length; ++i) {
-    var entry = array[i];
-    %_CallFunction(map, entry[0], entry[1], MapSet);
+  for (var i = 0; i + 1 < length; i += 2) {

[arv (Not doing code reviews), 2015/06/02 22:35:43]

This + 1 is not needed

[adamk, 2015/06/02 22:40:03]

Tough habit to break from C++, I'd rather leave it in if you don't mind...

[adamk, 2015/06/02 22:56:12]

Okay, changed my mind, removed the +1 and hardened the API to require an even
length.

+    var key = array[i];
+    var value = array[i + 1];
+    %_CallFunction(map, key, value, MapSet);
   }
   return map;
 };