Group all local functions at the top of the file and add comments

diff --git a/src/effects.js b/src/effects.js
index f334ac95b3a5746d9daf8890eea86158178f49e7..80c033abcdfb47d5a6fd2ce0d2f7d113dab21471 100644 (file)
--- a/src/effects.js
+++ b/src/effects.js
@@ -17,13 +17,44 @@ var elemdisplay = {},
            window.mozRequestAnimationFrame ||
            window.oRequestAnimationFrame;
 
+// Animations created synchronously will run synchronously
+function createFxNow() {
+       setTimeout( clearFxNow, 0 );
+       return ( fxNow = jQuery.now() );
+}
+
 function clearFxNow() {
        fxNow = undefined;
 }
 
-function createFxNow() {
-       setTimeout( clearFxNow, 0 );
-       return ( fxNow = jQuery.now() );
+// Try to restore the default display value of an element
+// fails if a display rule has been set for this element, e.g. div { display: inline; }
+function defaultDisplay( nodeName ) {
+       if ( !elemdisplay[ nodeName ] ) {
+               var elem = jQuery("<" + nodeName + ">").appendTo("body"),
+                       display = elem.css("display");
+
+               elem.remove();
+
+               if ( display === "none" || display === "" ) {
+                       display = "block";
+               }
+
+               elemdisplay[ nodeName ] = display;
+       }
+
+       return elemdisplay[ nodeName ];
+}
+
+// Generate parameters to create a standard animation
+function genFx( type, num ) {
+       var obj = {};
+
+       jQuery.each( fxAttrs.concat.apply([], fxAttrs.slice(0,num)), function() {
+               obj[ this ] = type;
+       });
+
+       return obj;
 }
 
 jQuery.fn.extend({
@@ -260,16 +291,6 @@ jQuery.fn.extend({
 
 });
 
-function genFx( type, num ) {
-       var obj = {};
-
-       jQuery.each( fxAttrs.concat.apply([], fxAttrs.slice(0,num)), function() {
-               obj[ this ] = type;
-       });
-
-       return obj;
-}
-
 // Generate shortcuts for custom animations
 jQuery.each({
        slideDown: genFx("show", 1),
@@ -536,21 +557,4 @@ if ( jQuery.expr && jQuery.expr.filters ) {
        };
 }
 
-function defaultDisplay( nodeName ) {
-       if ( !elemdisplay[ nodeName ] ) {
-               var elem = jQuery("<" + nodeName + ">").appendTo("body"),
-                       display = elem.css("display");
-
-               elem.remove();
-
-               if ( display === "none" || display === "" ) {
-                       display = "block";
-               }
-
-               elemdisplay[ nodeName ] = display;
-       }
-
-       return elemdisplay[ nodeName ];
-}
-
 })( jQuery );