Cosmetic and style cleanups.

diff --git a/src/event.js b/src/event.js
index fc43ed1ae268cf1bcb5034cabfce3cf1531c5992..c5c5ca66b19a0d93e2406d5f05b283461363adc7 100644 (file)
--- a/src/event.js
+++ b/src/event.js
@@ -42,9 +42,8 @@ var rnamespaces = /\.(.*)$/,
        };
 
 /*
- * A number of helper functions used for managing events.
- * Many of the ideas behind this code originated from
- * Dean Edwards' addEvent library.
+ * Helper functions for managing events -- not part of the public interface.
+ * Props to Dean Edwards' addEvent library for many of the ideas.
  */
 jQuery.event = {
 
@@ -275,7 +274,7 @@ jQuery.event = {
                        namespaces = [],
                        exclusive, i, cur, old, ontype, special, doc, eventPath, bubbleType,
                        addHandlers = function( elem, type ) {
-                               // Defer getting handler in case propagation is stopped
+                               // Defer getting handler so we don't waste time in case propagation is stopped
                                if ( (jQuery._data( elem, "events" ) || {})[ type ] ) {
                                        eventPath.push({ elem: elem, type: type /*, handler: jQuery._data( elem, "handle" ) */ });
                                }
@@ -285,13 +284,13 @@ jQuery.event = {
                                }
                        };
 
-               if ( type.indexOf("!") >= 0 ) {
+               if ( type.indexOf( "!" ) >= 0 ) {
                        // Exclusive events trigger only for the exact event (no namespaces)
                        type = type.slice(0, -1);
                        exclusive = true;
                }
 
-               if ( type.indexOf(".") >= 0 ) {
+               if ( type.indexOf( "." ) >= 0 ) {
                        // Namespaced trigger; create a regexp to match event type in handle()
                        namespaces = type.split(".");
                        type = namespaces.shift();
@@ -314,7 +313,7 @@ jQuery.event = {
 
                event.type = type;
                event.exclusive = exclusive;
-               event.namespace = namespaces.join(".");
+               event.namespace = namespaces.join( "." );
                event.namespace_re = event.namespace? new RegExp("(^|\\.)" + namespaces.join("\\.(?:.*\\.)?") + "(\\.|$)") : null;
                ontype = type.indexOf( ":" ) < 0 ? "on" + type : "";
 
@@ -458,8 +457,8 @@ jQuery.event = {
 
                // Run delegates first; they may want to stop propagation beneath us
                event.delegateTarget = this;
-               for ( i=0; i < handlerQueue.length && !event.isPropagationStopped(); i++ ) {
-                       matched = handlerQueue[i];
+               for ( i = 0; i < handlerQueue.length && !event.isPropagationStopped(); i++ ) {
+                       matched = handlerQueue[ i ];
                        dispatch( matched.elem, event, matched.matches, args );
                }
                delete event.delegateTarget;
@@ -744,7 +743,7 @@ if ( !jQuery.support.submitBubbles ) {
                                return false;
                        }
 
-                       jQuery.event.add(this, "click._submit keypress._submit", function( e ) {
+                       jQuery.event.add( this, "click._submit keypress._submit", function( e ) {
                                var elem = e.target,
                                        type = jQuery.nodeName( elem, "input" ) || jQuery.nodeName( elem, "button" ) ? elem.type : "";
 
@@ -780,7 +779,7 @@ if ( !jQuery.support.changeBubbles ) {
                        val = elem.selectedIndex > -1 ?
                                jQuery.map( elem.options, function( elem ) {
                                        return elem.selected;
-                               }).join("-") :
+                               }).join( "-" ) :
                                "";
                }
 
@@ -796,7 +795,7 @@ if ( !jQuery.support.changeBubbles ) {
                }
 
                old = jQuery._data( elem, "_change_data" );
-               val = getVal(elem);
+               val = getVal( elem );
 
                // the current data will be also retrieved by beforeactivate
                if ( e.type !== "focusout" || elem.type !== "radio" ) {
@@ -814,41 +813,41 @@ if ( !jQuery.support.changeBubbles ) {
                beforedeactivate: testChange,
 
                click: function( e ) {
-               var elem = e.target,
-                       name = elem.nodeName.toLowerCase(),
-                       type = name === "input"? elem.type : "";
+                       var elem = e.target,
+                               name = elem.nodeName.toLowerCase(),
+                               type = name === "input"? elem.type : "";
 
-               if ( type === "radio" || type === "checkbox" || name === "select" ) {
-                               testChange.call( this, e );
+                       if ( type === "radio" || type === "checkbox" || name === "select" ) {
+                                       testChange.call( this, e );
                        }
                },
 
                // Change has to be called before submit
                // Keydown will be called before keypress, which is used in submit-event delegation
                keydown: function( e ) {
-               var elem = e.target,
-                       name = elem.nodeName.toLowerCase(),
-                       type = name === "input"? elem.type : "";
-
-               if ( (e.keyCode === 13 && name !== "textarea") ||
-                               (e.keyCode === 32 && (type === "checkbox" || type === "radio")) ||
-                               type === "select-multiple" ) {
-                               testChange.call( this, e );
+                       var elem = e.target,
+                               name = elem.nodeName.toLowerCase(),
+                               type = name === "input"? elem.type : "";
+
+                       if ( (e.keyCode === 13 && name !== "textarea") ||
+                                       (e.keyCode === 32 && (type === "checkbox" || type === "radio")) ||
+                                       type === "select-multiple" ) {
+                                       testChange.call( this, e );
                        }
                },
 
                // Beforeactivate happens also before the previous element is blurred
                // here, you can't trigger a change event, but you can store data
-                       beforeactivate: function( e ) {
-                               var elem = e.target;
-                               jQuery._data( elem, "_change_data", getVal(elem) );
+               beforeactivate: function( e ) {
+                       var elem = e.target;
+                       jQuery._data( elem, "_change_data", getVal( elem ) );
                },
 
                // Update the current value if we're not re-focusing (#8157)
                focus: function( e ) {
                        var elem = e.target;
                        if ( elem != document.activeElement ) {
-                               jQuery._data( elem, "_change_data", getVal(elem) );
+                               jQuery._data( elem, "_change_data", getVal( elem ) );
                        }
                }
        };
@@ -861,7 +860,7 @@ if ( !jQuery.support.changeBubbles ) {
                        }
 
                        for ( var type in changeFilters ) {
-                               jQuery.event.add( this, type + "._change", changeFilters[type] );
+                               jQuery.event.add( this, type + "._change", changeFilters[ type ] );
                        }
 
                        return rformElems.test( this.nodeName );
@@ -929,7 +928,7 @@ jQuery.fn.extend({
                                selector = undefined;
                        }
                        for ( type in types ) {
-                               this.on( type, selector, data, types[type], one );
+                               this.on( type, selector, data, types[ type ], one );
                        }
                        return this;
                }
@@ -963,7 +962,7 @@ jQuery.fn.extend({
                                return origFn.apply( this, arguments );
                        };
                        // Use same guid so caller can remove using origFn
-                       fn.guid = origFn.guid || (origFn.guid = jQuery.guid++);
+                       fn.guid = origFn.guid || ( origFn.guid = jQuery.guid++ );
                }
                return this.each( function() {
                        jQuery.event.add( this, types, fn, data, selector );
@@ -980,7 +979,7 @@ jQuery.fn.extend({
                if ( typeof types === "object" ) {
                        // ( types-object [, selector] )
                        for ( var type in types ) {
-                               this.off( type, selector, types[type] );
+                               this.off( type, selector, types[ type ] );
                        }
                        return this;
                }