Ticket #6919: array.patch

array.js

@@ -65 +65 @@
                         // FIXME: there are several ways of handilng thisObject. Is
                         // dojo.global always the default context?
                         var _p = _getParts(arr, thisObject, callback); arr = _p[0];
-                        for(var i=0,l=_p[0].length; i<l; i++){ 
+                        for(var i=0,l=arr.length; i<l; ++i){ 
                                 _p[2].call(_p[1], arr[i], i, arr);
                         }
                 },
 
                 _everyOrSome: function(/*Boolean*/every, /*Array|String*/arr, /*Function|String*/callback, /*Object?*/thisObject){
                         var _p = _getParts(arr, thisObject, callback); arr = _p[0];
-                        for(var i = 0, l = arr.length; i < l; i++){
+                        for(var i=0,l=arr.length; i<l; ++i){
                                 var result = !!_p[2].call(_p[1], arr[i], i, arr);
                                 if(every ^ result){
                                         return result; // Boolean
@@ -142 +142 @@
                         //              returns [2, 3, 4, 5]
                         var _p = _getParts(arr, thisObject, callback); arr = _p[0];
                         var outArr = (arguments[3] ? (new arguments[3]()) : []);
-                        for(var i=0;i<arr.length;++i){
+                        for(var i=0,l=arr.length; i<l; ++i){
                                 outArr.push(_p[2].call(_p[1], arr[i], i, arr));
                         }
                         return outArr; // Array
@@ -167 +167 @@
 
                         var _p = _getParts(arr, thisObject, callback); arr = _p[0];
                         var outArr = [];
-                        for(var i = 0; i < arr.length; i++){
+                        for(var i=0,l=arr.length; i<l; ++i){
                                 if(_p[2].call(_p[1], arr[i], i, arr)){
                                         outArr.push(arr[i]);
                                 }