Fix some code style inconsistencies

diff --git a/src/ajax.js b/src/ajax.js
index aa3bab34831e9497325134c5a69349d0a7da7527..174e8274dafab0ed1b2aa6449060f6f7840b4559 100644 (file)
--- a/src/ajax.js
+++ b/src/ajax.js
@@ -102,7 +102,7 @@ function inspectPrefiltersOrTransports( structure, options, originalOptions, jqX
                inspected[ dataType ] = true;
                jQuery.each( structure[ dataType ] || [], function( _, prefilterOrFactory ) {
                        var dataTypeOrTransport = prefilterOrFactory( options, originalOptions, jqXHR );
-                       if( typeof dataTypeOrTransport === "string" && !seekingTransport && !inspected[ dataTypeOrTransport ] ) {
+                       if ( typeof dataTypeOrTransport === "string" && !seekingTransport && !inspected[ dataTypeOrTransport ] ) {
                                options.dataTypes.unshift( dataTypeOrTransport );
                                inspect( dataTypeOrTransport );
                                return false;
@@ -135,7 +135,6 @@ function ajaxExtend( target, src ) {
        return target;
 }
 
-
 /* Handles responses to an ajax request:
  * - finds the right dataType (mediates between content-type and expected dataType)
  * - returns the corresponding response
@@ -146,7 +145,7 @@ function ajaxHandleResponses( s, jqXHR, responses ) {
                dataTypes = s.dataTypes;
 
        // Remove auto dataType and get content-type in the process
-       while( dataTypes[ 0 ] === "*" ) {
+       while ( dataTypes[ 0 ] === "*" ) {
                dataTypes.shift();
                if ( ct === undefined ) {
                        ct = s.mimeType || jqXHR.getResponseHeader("Content-Type");
@@ -512,7 +511,7 @@ jQuery.extend({
                s.type = options.method || options.type || s.method || s.type;
 
                // Extract dataTypes list
-               s.dataTypes = jQuery.trim( s.dataType || "*" ).toLowerCase().match( rnotwhite ) || [""];
+               s.dataTypes = jQuery.trim( s.dataType || "*" ).toLowerCase().match( rnotwhite ) || [ "" ];
 
                // A cross-domain request is in order when we have a protocol:host:port mismatch
                if ( s.crossDomain == null ) {
@@ -798,8 +797,8 @@ jQuery.each( [ "get", "post" ], function( i, method ) {
 });
 
 // Attach a bunch of functions for handling common AJAX events
-jQuery.each( [ "ajaxStart", "ajaxStop", "ajaxComplete", "ajaxError", "ajaxSuccess", "ajaxSend" ], function( i, type ){
-       jQuery.fn[ type ] = function( fn ){
+jQuery.each( [ "ajaxStart", "ajaxStop", "ajaxComplete", "ajaxError", "ajaxSuccess", "ajaxSend" ], function( i, type ) {
+       jQuery.fn[ type ] = function( fn ) {
                return this.on( type, fn );
        };
 });

diff --git a/src/ajax/parseXML.js b/src/ajax/parseXML.js
index a987f57fc5f3bf635f90f9b7a07bf6a6b7fced6a..0b92c7a42bfc97ef957a89fb8575fe2f03b19646 100644 (file)
--- a/src/ajax/parseXML.js
+++ b/src/ajax/parseXML.js
@@ -11,7 +11,7 @@ jQuery.parseXML = function( data ) {
        try {
                if ( window.DOMParser ) { // Standard
                        tmp = new DOMParser();
-                       xml = tmp.parseFromString( data , "text/xml" );
+                       xml = tmp.parseFromString( data, "text/xml" );
                } else { // IE
                        xml = new ActiveXObject( "Microsoft.XMLDOM" );
                        xml.async = "false";

diff --git a/src/attributes/support.js b/src/attributes/support.js
index aeeb3af2f915df361c2e269823f0abb9f63847a3..230722499cd62e5d17061f726c88896ca165ae4b 100644 (file)
--- a/src/attributes/support.js
+++ b/src/attributes/support.js
@@ -2,7 +2,7 @@ define([
        "../var/support"
 ], function( support ) {
 
-(function () {
+(function() {
        var a, input, select, opt,
                div = document.createElement("div" );
 

diff --git a/src/attributes/val.js b/src/attributes/val.js
index 1b5e86e1e7e705e23e59fa85ae5edbcaac13add4..8454e80d038d1a4c2eb00076f87a046758859153 100644 (file)
--- a/src/attributes/val.js
+++ b/src/attributes/val.js
@@ -52,7 +52,7 @@ jQuery.fn.extend({
                        } else if ( typeof val === "number" ) {
                                val += "";
                        } else if ( jQuery.isArray( val ) ) {
-                               val = jQuery.map(val, function ( value ) {
+                               val = jQuery.map( val, function( value ) {
                                        return value == null ? "" : value + "";
                                });
                        }

diff --git a/src/callbacks.js b/src/callbacks.js
index 7090384acf236816cf0226826e969dc3fd7a1807..9d12823d90d505b434c351ee491b2d52bb3e829c 100644 (file)
--- a/src/callbacks.js
+++ b/src/callbacks.js
@@ -126,7 +126,7 @@ jQuery.Callbacks = function( options ) {
                                if ( list ) {
                                        jQuery.each( arguments, function( _, arg ) {
                                                var index;
-                                               while( ( index = jQuery.inArray( arg, list, index ) ) > -1 ) {
+                                               while ( ( index = jQuery.inArray( arg, list, index ) ) > -1 ) {
                                                        list.splice( index, 1 );
                                                        // Handle firing indexes
                                                        if ( firing ) {

diff --git a/src/core/ready.js b/src/core/ready.js
index 87be241550cef1acef159f42bf0110c8cea1e029..3d0510e38ad625228779ad4c4007b110cf7041de 100644 (file)
--- a/src/core/ready.js
+++ b/src/core/ready.js
@@ -87,7 +87,6 @@ function completed() {
        }
 }
 
-
 jQuery.ready.promise = function( obj ) {
        if ( !readyList ) {
 

diff --git a/src/css.js b/src/css.js
index b18f8220212630f757a4d53add90d0141a237ea1..0f90f0b6a0ad9aa2d0d45946b2fbb827c1927c92 100644 (file)
--- a/src/css.js
+++ b/src/css.js
@@ -422,7 +422,7 @@ if ( !support.opacity ) {
 }
 
 jQuery.cssHooks.marginRight = addGetHookIf( support.reliableMarginRight,
-       function ( elem, computed ) {
+       function( elem, computed ) {
                if ( computed ) {
                        // WebKit Bug 13343 - getComputedStyle returns wrong value for margin-right
                        // Work around by temporarily setting element display to inline-block

diff --git a/src/css/support.js b/src/css/support.js
index fdc1dbd1246c9fc9e1ea3f24d0f239ed393d72d9..d5ea928203c40d8d21c73daae790a2cbb757acef 100644 (file)
--- a/src/css/support.js
+++ b/src/css/support.js
@@ -3,7 +3,7 @@ define([
        "../var/support"
 ], function( jQuery, support ) {
 
-(function () {
+(function() {
        var a, reliableHiddenOffsetsVal, boxSizingVal, boxSizingReliableVal,
                pixelPositionVal, reliableMarginRightVal,
                div = document.createElement( "div" ),
@@ -34,7 +34,6 @@ define([
        // Null elements to avoid leaks in IE.
        a = div = null;
 
-
        jQuery.extend(support, {
                reliableHiddenOffsets: function() {
                        if ( reliableHiddenOffsetsVal != null ) {

diff --git a/src/css/var/rmargin.js b/src/css/var/rmargin.js
index 7597cd31194950cf13c0c3168aa41abef4e680b3..da0438db675de626efbe2cecb9134ae091c811e9 100644 (file)
--- a/src/css/var/rmargin.js
+++ b/src/css/var/rmargin.js
@@ -1,3 +1,3 @@
 define(function() {
        return (/^margin/);
-});
\ No newline at end of file
+});

diff --git a/src/css/var/rnumnonpx.js b/src/css/var/rnumnonpx.js
index ec4fd615c33c1ba64149c87234c1597f7456d7de..c93be285061daa2b5bf65931c5b69d84a58910a4 100644 (file)
--- a/src/css/var/rnumnonpx.js
+++ b/src/css/var/rnumnonpx.js
@@ -2,4 +2,4 @@ define([
        "../../var/pnum"
 ], function( pnum ) {
        return new RegExp( "^(" + pnum + ")(?!px)[a-z%]+$", "i" );
-});
\ No newline at end of file
+});

diff --git a/src/data.js b/src/data.js
index 256bed7cd689ba94329b11fc4990b112198c5680..03994cdb393927a4298cab0ebfa49ceeb1e3bcd0 100644 (file)
--- a/src/data.js
+++ b/src/data.js
@@ -56,7 +56,7 @@ function isEmptyDataObject( obj ) {
        return true;
 }
 
-function internalData( elem, name, data, pvt /* Internal Use Only */ ){
+function internalData( elem, name, data, pvt /* Internal Use Only */ ) {
        if ( !jQuery.acceptData( elem ) ) {
                return;
        }

diff --git a/src/data/support.js b/src/data/support.js
index 52250b21d85d2a5775608ca3d77f6175a11cb3e4..e57f6f7de77405342efd9b48ac5cb53abfaf3f6f 100644 (file)
--- a/src/data/support.js
+++ b/src/data/support.js
@@ -2,7 +2,7 @@ define([
        "../var/support"
 ], function( support ) {
 
-(function () {
+(function() {
        var div = document.createElement( "div" );
 
        // Execute the test only if not already executed in another module.

diff --git a/src/deferred.js b/src/deferred.js
index 1c2f87e868ccb02b71231bb5a6d06ae869847992..8dedbf74c0cc97f1e510f90b6734156ee01318b8 100644 (file)
--- a/src/deferred.js
+++ b/src/deferred.js
@@ -109,9 +109,10 @@ jQuery.extend({
                                return function( value ) {
                                        contexts[ i ] = this;
                                        values[ i ] = arguments.length > 1 ? slice.call( arguments ) : value;
-                                       if( values === progressValues ) {
+                                       if ( values === progressValues ) {
                                                deferred.notifyWith( contexts, values );
-                                       } else if ( !( --remaining ) ) {
+
+                                       } else if ( !(--remaining) ) {
                                                deferred.resolveWith( contexts, values );
                                        }
                                };

diff --git a/src/effects.js b/src/effects.js
index 582870fe93543b80bd3379eecfc195e1a8078001..bd2e4eef944139e33e68a329a05d14e4add6ca67 100644 (file)
--- a/src/effects.js
+++ b/src/effects.js
@@ -21,7 +21,7 @@ var
        rrun = /queueHooks$/,
        animationPrefilters = [ defaultPrefilter ],
        tweeners = {
-               "*": [function( prop, value ) {
+               "*": [ function( prop, value ) {
                        var tween = this.createTween( prop, value ),
                                target = tween.cur(),
                                parts = rfxnum.exec( value ),
@@ -68,7 +68,7 @@ var
                        }
 
                        return tween;
-               }]
+               } ]
        };
 
 // Animations created synchronously will run synchronously
@@ -87,8 +87,8 @@ function genFx( type, includeWidth ) {
 
        // if we include width, step value is 1 to do all cssExpand values,
        // if we don't include width, step value is 2 to skip over Left and Right
-       includeWidth = includeWidth? 1 : 0;
-       for( ; i < 4 ; i += 2 - includeWidth ) {
+       includeWidth = includeWidth ? 1 : 0;
+       for ( ; i < 4 ; i += 2 - includeWidth ) {
                which = cssExpand[ i ];
                attrs[ "margin" + which ] = attrs[ "padding" + which ] = type;
        }
@@ -100,7 +100,6 @@ function genFx( type, includeWidth ) {
        return attrs;
 }
 
-
 function createTween( value, prop, animation ) {
        var tween,
                collection = ( tweeners[ prop ] || [] ).concat( tweeners[ "*" ] ),
@@ -189,7 +188,6 @@ function defaultPrefilter( elem, props, opts ) {
                }
        }
 
-
        // show/hide pass
        for ( prop in props ) {
                value = props[ prop ];

diff --git a/src/effects/Tween.js b/src/effects/Tween.js
index 43779b0651954e7ea10bdb32c756df1dad07e011..12eec55cfc00ca558528c10cc4b130c4801e4e50 100644 (file)
--- a/src/effects/Tween.js
+++ b/src/effects/Tween.js
@@ -102,7 +102,7 @@ jQuery.easing = {
                return p;
        },
        swing: function( p ) {
-               return 0.5 - Math.cos( p*Math.PI ) / 2;
+               return 0.5 - Math.cos( p * Math.PI ) / 2;
        }
 };
 

diff --git a/src/effects/support.js b/src/effects/support.js
index 0299cae4ed260b2d4ea067d38736943879111e48..2391bcbffa4c63bc20fa006a2d25dce960490d0b 100644 (file)
--- a/src/effects/support.js
+++ b/src/effects/support.js
@@ -3,7 +3,7 @@ define([
        "../var/support"
 ], function( strundefined, support ) {
 
-(function () {
+(function() {
        var a, shrinkWrapBlocksVal,
                div = document.createElement( "div" ),
                divReset =
@@ -32,7 +32,7 @@ define([
        // Null elements to avoid leaks in IE.
        a = div = null;
 
-       support.shrinkWrapBlocks = function () {
+       support.shrinkWrapBlocks = function() {
                var body, container, div, containerStyles;
 
                if ( shrinkWrapBlocksVal == null ) {

diff --git a/src/event.js b/src/event.js
index a6f69f49ff505972c8963941be692302230a7771..2ca50aba45ca691de3d3b41600e0e97fdc5e4338 100644 (file)
--- a/src/event.js
+++ b/src/event.js
@@ -79,7 +79,7 @@ jQuery.event = {
                }
 
                // Handle multiple events separated by a space
-               types = ( types || "" ).match( rnotwhite ) || [""];
+               types = ( types || "" ).match( rnotwhite ) || [ "" ];
                t = types.length;
                while ( t-- ) {
                        tmp = rtypenamespace.exec( types[t] ) || [];
@@ -165,7 +165,7 @@ jQuery.event = {
                }
 
                // Once for each type.namespace in types; type may be omitted
-               types = ( types || "" ).match( rnotwhite ) || [""];
+               types = ( types || "" ).match( rnotwhite ) || [ "" ];
                t = types.length;
                while ( t-- ) {
                        tmp = rtypenamespace.exec( types[t] ) || [];

diff --git a/src/event/support.js b/src/event/support.js
index d2a092250a704f2ff8711124d9ca77f286efdaf0..caac5179f38cd96ff54f890d5dd244a9e7c6a3ff 100644 (file)
--- a/src/event/support.js
+++ b/src/event/support.js
@@ -2,7 +2,7 @@ define([
        "../var/support"
 ], function( support ) {
 
-(function () {
+(function() {
        var i, eventName,
                div = document.createElement( "div" );
 

diff --git a/src/manipulation.js b/src/manipulation.js
index 08a9b7c04c446c16c63fb500db66c1d8027792c3..ee0baeb7ce47c4a82bd35eec9957b985a0959100 100644 (file)
--- a/src/manipulation.js
+++ b/src/manipulation.js
@@ -311,7 +311,7 @@ jQuery.extend({
                                        tmp = tmp || safe.appendChild( context.createElement("div") );
 
                                        // Deserialize a standard representation
-                                       tag = ( rtagName.exec( elem ) || ["", ""] )[1].toLowerCase();
+                                       tag = (rtagName.exec( elem ) || [ "", "" ])[ 1 ].toLowerCase();
                                        wrap = wrapMap[ tag ] || wrapMap._default;
 
                                        tmp.innerHTML = wrap[1] + elem.replace( rxhtmlTag, "<$1></$2>" ) + wrap[2];
@@ -556,7 +556,7 @@ jQuery.fn.extend({
                dataAndEvents = dataAndEvents == null ? false : dataAndEvents;
                deepDataAndEvents = deepDataAndEvents == null ? dataAndEvents : deepDataAndEvents;
 
-               return this.map( function () {
+               return this.map(function() {
                        return jQuery.clone( this, dataAndEvents, deepDataAndEvents );
                });
        },
@@ -577,7 +577,7 @@ jQuery.fn.extend({
                        if ( typeof value === "string" && !rnoInnerhtml.test( value ) &&
                                ( support.htmlSerialize || !rnoshimcache.test( value )  ) &&
                                ( support.leadingWhitespace || !rleadingWhitespace.test( value ) ) &&
-                               !wrapMap[ ( rtagName.exec( value ) || ["", ""] )[1].toLowerCase() ] ) {
+                               !wrapMap[ (rtagName.exec( value ) || [ "", "" ])[ 1 ].toLowerCase() ] ) {
 
                                value = value.replace( rxhtmlTag, "<$1></$2>" );
 

diff --git a/src/manipulation/support.js b/src/manipulation/support.js
index fd719274e358c4a3466b634ce5863ff5e73da146..427660361e4fc3f19c7efae5739ab136212afe22 100644 (file)
--- a/src/manipulation/support.js
+++ b/src/manipulation/support.js
@@ -2,7 +2,7 @@ define([
        "../var/support"
 ], function( support ) {
 
-(function () {
+(function() {
        var fragment = document.createDocumentFragment(),
                div = document.createElement("div"),
                input = document.createElement("input");

diff --git a/src/offset.js b/src/offset.js
index 8b6bacd3fe27cf2e78bfbfa473cd31bbe0f0d67a..0d35a4f09ef33f6ffeffb4977fe42b0edb08ae19 100644 (file)
--- a/src/offset.js
+++ b/src/offset.js
@@ -44,7 +44,7 @@ jQuery.offset = {
                curCSSTop = jQuery.css( elem, "top" );
                curCSSLeft = jQuery.css( elem, "left" );
                calculatePosition = ( position === "absolute" || position === "fixed" ) &&
-                       jQuery.inArray("auto", [curCSSTop, curCSSLeft]) > -1;
+                       jQuery.inArray("auto", [ curCSSTop, curCSSLeft ] ) > -1;
 
                // need to be able to calculate position if either top or left is auto and position is either absolute or fixed
                if ( calculatePosition ) {
@@ -163,7 +163,7 @@ jQuery.fn.extend({
 });
 
 // Create scrollLeft and scrollTop methods
-jQuery.each( {scrollLeft: "pageXOffset", scrollTop: "pageYOffset"}, function( method, prop ) {
+jQuery.each( { scrollLeft: "pageXOffset", scrollTop: "pageYOffset" }, function( method, prop ) {
        var top = /Y/.test( prop );
 
        jQuery.fn[ method ] = function( val ) {
@@ -195,7 +195,7 @@ jQuery.each( {scrollLeft: "pageXOffset", scrollTop: "pageYOffset"}, function( me
 // rather than make the css module depend on the offset module, we just check for it here
 jQuery.each( [ "top", "left" ], function( i, prop ) {
        jQuery.cssHooks[ prop ] = addGetHookIf( support.pixelPosition,
-               function ( elem, computed ) {
+               function( elem, computed ) {
                        if ( computed ) {
                                computed = curCSS( elem, prop );
                                // if curCSS returns percentage, fallback to offset

diff --git a/src/queue.js b/src/queue.js
index f5a1b7467b4f3b3614db39d41282050eddb5808b..29e766078be5fa8c7067d8f6b9f5a18fed5ab403 100644 (file)
--- a/src/queue.js
+++ b/src/queue.js
@@ -126,7 +126,7 @@ jQuery.fn.extend({
                }
                type = type || "fx";
 
-               while( i-- ) {
+               while ( i-- ) {
                        tmp = jQuery._data( elements[ i ], type + "queueHooks" );
                        if ( tmp && tmp.empty ) {
                                count++;

diff --git a/src/selector-sizzle.js b/src/selector-sizzle.js
index 3dbfa3cd072020a198470aa800d725e3908d753c..7d3926b1a989436e07b8815e852eda32f6ba4f60 100644 (file)
--- a/src/selector-sizzle.js
+++ b/src/selector-sizzle.js
@@ -1,7 +1,7 @@
 define([
        "./core",
        "sizzle"
-], function ( jQuery, Sizzle ) {
+], function( jQuery, Sizzle ) {
 
 jQuery.find = Sizzle;
 jQuery.expr = Sizzle.selectors;

diff --git a/src/serialize.js b/src/serialize.js
index 797d735aaf64d125edf3c66a655f381cbeff8c00..5b09cb6145113974542b2479d498db4dcbd49750 100644 (file)
--- a/src/serialize.js
+++ b/src/serialize.js
@@ -80,25 +80,25 @@ jQuery.fn.extend({
                return jQuery.param( this.serializeArray() );
        },
        serializeArray: function() {
-               return this.map(function(){
+               return this.map(function() {
                        // Can add propHook for "elements" to filter or add form elements
                        var elements = jQuery.prop( this, "elements" );
                        return elements ? jQuery.makeArray( elements ) : this;
                })
-               .filter(function(){
+               .filter(function() {
                        var type = this.type;
                        // Use .is(":disabled") so that fieldset[disabled] works
                        return this.name && !jQuery( this ).is( ":disabled" ) &&
                                rsubmittable.test( this.nodeName ) && !rsubmitterTypes.test( type ) &&
                                ( this.checked || !rcheckableType.test( type ) );
                })
-               .map(function( i, elem ){
+               .map(function( i, elem ) {
                        var val = jQuery( this ).val();
 
                        return val == null ?
                                null :
                                jQuery.isArray( val ) ?
-                                       jQuery.map( val, function( val ){
+                                       jQuery.map( val, function( val ) {
                                                return { name: elem.name, value: val.replace( rCRLF, "\r\n" ) };
                                        }) :
                                        { name: elem.name, value: val.replace( rCRLF, "\r\n" ) };