Skip to content

Commit 6e61c0a

Browse files
committed
Storing object[ name ] in a local variable to save property lookup times in jQuery.each when iterating over objects.
1 parent 1533bf7 commit 6e61c0a

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/core.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -535,7 +535,7 @@ jQuery.extend({
535535

536536
// args is for internal usage only
537537
each: function( object, callback, args ) {
538-
var name, i = 0,
538+
var name, value, i = 0,
539539
length = object.length,
540540
isObj = length === undefined || jQuery.isFunction(object);
541541

@@ -558,12 +558,13 @@ jQuery.extend({
558558
} else {
559559
if ( isObj ) {
560560
for ( name in object ) {
561-
if ( callback.call( object[ name ], name, object[ name ] ) === false ) {
561+
value = object[ name ];
562+
if ( callback.call( value, name, value ) === false ) {
562563
break;
563564
}
564565
}
565566
} else {
566-
for ( var value = object[0];
567+
for ( value = object[0];
567568
i < length && callback.call( value, i, value ) !== false; value = object[++i] ) {}
568569
}
569570
}

0 commit comments

Comments
 (0)