Changeset 2790

Timestamp:

08/20/07 07:04:00 (1 year ago)

Author:

jeresig

Message:

Just pushed in my changes for making jQuery.each on objects faster, sample results: http://dev.jquery.com/~john/ticket/each/obj.html

Files:

• trunk/jquery/src/jquery/jquery.js (modified) (1 diff)

trunk/jquery/src/jquery/jquery.js

@@ -1392 +1392 @@
     // args is for internal usage only
     each: function( obj, fn, args ) {
-        if ( obj.length == undefined )
-            for ( var i in obj )
-                fn.apply( obj[i], args || [i, obj[i]] );
-        else if ( args ) {
-            for ( var i = 0, ol = obj.length; i < ol; i++ )
-                if ( fn.apply( obj[i], args ) === false ) break;
+        if ( args ) {
+            if ( obj.length == undefined )
+                for ( var i in obj )
+                    fn.apply( obj[i], args );
+            else
+                for ( var i = 0, ol = obj.length; i < ol; i++ )
+                    if ( fn.apply( obj[i], args ) === false ) break;
 
         // A special, fast, case for the most common use of each
-        } else
-            for ( var i = 0, ol = obj.length, val = obj[0]; 
-                i < ol && fn.call(val,i,val) !== false; val = obj[++i] );
+        } else {
+            if ( obj.length == undefined )
+                for ( var i in obj )
+                    fn.call( obj[i], i, obj[i] );
+            else
+                for ( var i = 0, ol = obj.length, val = obj[0]; 
+                    i < ol && fn.call(val,i,val) !== false; val = obj[++i] );
+        }
 
         return obj;