Changeset 5570

Timestamp:

05/12/08 19:45:02 (7 months ago)

Author:

aflesler

Message:

jquery fx & event: replaced the use of .apply(), where .call() could had been used.

Location:

trunk/jquery/src

Files:

• event.js (modified) (1 diff)
• fx.js (modified) (6 diffs)

trunk/jquery/src/event.js

@@ -496 +496 @@
                 // Execute all of them
                 jQuery.each( jQuery.readyList, function(){
-                    this.apply( document );
+                    this.call( document );
                 });
                 

trunk/jquery/src/fx.js

@@ -82 +82 @@
             for ( p in prop ) {
                 if ( prop[p] == "hide" && hidden || prop[p] == "show" && !hidden )
-                    return jQuery.isFunction(opt.complete) && opt.complete.apply(this);
+                    return jQuery.isFunction(opt.complete) && opt.complete.call(this);
 
                 if ( p == "height" || p == "width" ) {
@@ -149 +149 @@
             
                 if ( queue(this, type).length == 1 )
-                    fn.apply(this);
+                    fn.call(this);
             }
         });
@@ -203 +203 @@
 
         if ( q.length )
-            q[0].apply( this );
+            q[0].call( this );
     });
 };
@@ -227 +227 @@
                 jQuery(this).dequeue();
             if ( jQuery.isFunction( opt.old ) )
-                opt.old.apply( this );
+                opt.old.call( this );
         };
     
@@ -261 +261 @@
     update: function(){
         if ( this.options.step )
-            this.options.step.apply( this.elem, [ this.now, this ] );
+            this.options.step.call( this.elem, this.now, this );
 
         (jQuery.fx.step[this.prop] || jQuery.fx.step._default)( this );
@@ -382 +382 @@
             if ( done && jQuery.isFunction( this.options.complete ) )
                 // Execute the complete function
-                this.options.complete.apply( this.elem );
+                this.options.complete.call( this.elem );
 
             return false;