From: jaubourg <j@ubourg.net>
Date: Sun, 23 Jan 2011 19:46:09 +0000 (+0100)
Subject: More code style fixes.
X-Git-Tag: 1.5rc1~8
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=ee22c8b34bb801f26cacc67a1a3a0ac595fe7928;p=jquery.git

More code style fixes.
---

diff --git a/src/ajax.js b/src/ajax.js
index 470e02254..bc3696ee8 100644
--- a/src/ajax.js
+++ b/src/ajax.js
@@ -255,7 +255,7 @@ jQuery.extend({
 
 		// If options is not an object,
 		// we simulate pre-1.5 signature
-		if ( typeof( options ) !== "object" ) {
+		if ( typeof options !== "object" ) {
 			options = url;
 			url = undefined;
 		}
@@ -423,7 +423,7 @@ jQuery.extend({
 			statusCode = undefined;
 
 			if ( s.global ) {
-				globalEventContext.trigger( "ajax" + ( isSuccess ? "Success" : "Error" ) ,
+				globalEventContext.trigger( "ajax" + ( isSuccess ? "Success" : "Error" ),
 						[ jXHR, s, isSuccess ? success : error ] );
 			}
 
@@ -455,7 +455,7 @@ jQuery.extend({
 					}
 				} else {
 					tmp = map[ jXHR.status ];
-					jXHR.then( tmp , tmp );
+					jXHR.then( tmp, tmp );
 				}
 			}
 			return this;
@@ -702,7 +702,7 @@ function inspectPrefiltersOrTransports( structure, options, originalOptions, dat
 			selected = list[ i ]( options, originalOptions );
 			// If we got redirected to a different dataType,
 			// we add it and switch to the corresponding list
-			if ( typeof( selected ) === "string" && selected !== dataType ) {
+			if ( typeof selected === "string" && selected !== dataType ) {
 				options.dataTypes.unshift( selected );
 				selected = inspectPrefiltersOrTransports(
 						structure, options, originalOptions, selected, tested );
@@ -757,7 +757,7 @@ function prefiltersOrTransports( structure, arg1, arg2, type /* internal */ ) {
 		return inspectPrefiltersOrTransports( structure, arg1, arg2 );
 	} else {
 		// We're requested to add to the structure
-		// Signature is ( dataTypeExpression , function )
+		// Signature is ( dataTypeExpression, function )
 		// with dataTypeExpression being optional and
 		// defaulting to auto ("*")
 		type = ( type === "function" );
@@ -818,11 +818,11 @@ function ajaxHandleResponses( s, jXHR, responses ) {
 	} else {
 		// Try convertible dataTypes
 		for ( type in responses ) {
-			if ( ! dataTypes[ 0 ] || s.converters[ type + " " + dataTypes[0] ] ) {
+			if ( !dataTypes[ 0 ] || s.converters[ type + " " + dataTypes[0] ] ) {
 				finalDataType = type;
 				break;
 			}
-			if ( ! firstDataType ) {
+			if ( !firstDataType ) {
 				firstDataType = type;
 			}
 		}
diff --git a/src/ajax/jsonp.js b/src/ajax/jsonp.js
index 0588475b6..f5742d998 100644
--- a/src/ajax/jsonp.js
+++ b/src/ajax/jsonp.js
@@ -12,10 +12,9 @@ jQuery.ajaxSetup({
 });
 
 // Detect, normalize options and install callbacks for jsonp requests
-// (dataIsString is used internally)
-jQuery.ajaxPrefilter( "json jsonp", function( s, originalSettings, dataIsString ) {
+jQuery.ajaxPrefilter( "json jsonp", function( s, originalSettings, dataIsString /* internal */ ) {
 
-	dataIsString = ( typeof( s.data ) === "string" );
+	dataIsString = ( typeof s.data === "string" );
 
 	if ( s.dataTypes[ 0 ] === "jsonp" ||
 		originalSettings.jsonpCallback ||
diff --git a/src/ajax/script.js b/src/ajax/script.js
index 37bd85b56..dfee86712 100644
--- a/src/ajax/script.js
+++ b/src/ajax/script.js
@@ -14,7 +14,7 @@ jQuery.ajaxSetup({
 });
 
 // Handle cache's special case and global
-jQuery.ajaxPrefilter( "script", function(s) {
+jQuery.ajaxPrefilter( "script", function( s ) {
 	if ( s.cache === undefined ) {
 		s.cache = false;
 	}