From: Mike Sherov <mike.sherov@gmail.com>
Date: Wed, 17 Oct 2012 16:50:12 +0000 (-0400)
Subject: no ticket: remove and enforce unused vars in jshint
X-Git-Tag: 1.9.0b1~181
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=16d996ba0c150101632cef140116b7f68da162f8;p=jquery.git

no ticket: remove and enforce unused vars in jshint
---

diff --git a/src/.jshintrc b/src/.jshintrc
index 2ce0eca6f..d74bf14fd 100644
--- a/src/.jshintrc
+++ b/src/.jshintrc
@@ -11,6 +11,7 @@
     "sub": true,
     "trailing": true,
     "undef": true,
+    "unused": true,
     "wsh": true
   },
   "globals": {
diff --git a/src/core.js b/src/core.js
index 709467168..5564a9870 100644
--- a/src/core.js
+++ b/src/core.js
@@ -81,7 +81,7 @@ var
 jQuery.fn = jQuery.prototype = {
 	constructor: jQuery,
 	init: function( selector, context, rootjQuery ) {
-		var match, elem, ret, doc;
+		var match, elem, doc;
 
 		// Handle $(""), $(null), $(undefined), $(false)
 		if ( !selector ) {
diff --git a/src/css.js b/src/css.js
index 6f5456315..ad42a048a 100644
--- a/src/css.js
+++ b/src/css.js
@@ -18,9 +18,7 @@ var curCSS, iframe, iframeDoc,
 	},
 
 	cssExpand = [ "Top", "Right", "Bottom", "Left" ],
-	cssPrefixes = [ "Webkit", "O", "Moz", "ms" ],
-
-	eventsToggle = jQuery.fn.toggle;
+	cssPrefixes = [ "Webkit", "O", "Moz", "ms" ];
 
 // return a css property mapped to a potentially vendor prefixed property
 function vendorPropName( style, name ) {
@@ -113,7 +111,7 @@ jQuery.fn.extend({
 	hide: function() {
 		return showHide( this );
 	},
-	toggle: function( state, fn2 ) {
+	toggle: function( state ) {
 		var bool = typeof state === "boolean";
 
 		return this.each(function() {
diff --git a/src/effects.js b/src/effects.js
index cff0dc835..dcc6c00b7 100644
--- a/src/effects.js
+++ b/src/effects.js
@@ -73,7 +73,6 @@ function createTweens( animation, props ) {
 function Animation( elem, properties, options ) {
 	var result,
 		index = 0,
-		tweenerIndex = 0,
 		length = animationPrefilters.length,
 		deferred = jQuery.Deferred().always( function() {
 			// don't match elem in the :animated selector
@@ -108,7 +107,7 @@ function Animation( elem, properties, options ) {
 			startTime: fxNow || createFxNow(),
 			duration: options.duration,
 			tweens: [],
-			createTween: function( prop, end, easing ) {
+			createTween: function( prop, end ) {
 				var tween = jQuery.Tween( elem, animation.opts, prop, end,
 						animation.opts.specialEasing[ prop ] || animation.opts.easing );
 				animation.tweens.push( tween );
diff --git a/src/event.js b/src/event.js
index f68a09c54..8e0f59afc 100644
--- a/src/event.js
+++ b/src/event.js
@@ -357,7 +357,7 @@ jQuery.event = {
 		// Make a writable jQuery.Event from the native event object
 		event = jQuery.event.fix( event || window.event );
 
-		var i, j, cur, ret, selMatch, matched, matches, handleObj, sel, related,
+		var i, j, cur, ret, selMatch, matched, matches, handleObj, sel,
 			handlers = ( (jQuery._data( this, "events" ) || {} )[ event.type ] || []),
 			delegateCount = handlers.delegateCount,
 			args = core_slice.call( arguments ),
@@ -704,8 +704,7 @@ jQuery.each({
 			var ret,
 				target = this,
 				related = event.relatedTarget,
-				handleObj = event.handleObj,
-				selector = handleObj.selector;
+				handleObj = event.handleObj;
 
 			// For mousenter/leave call the handler if related is outside the target.
 			// NB: No relatedTarget if the mouse left/entered the browser window
diff --git a/src/manipulation.js b/src/manipulation.js
index 7d18d04c9..ef305032e 100644
--- a/src/manipulation.js
+++ b/src/manipulation.js
@@ -650,7 +650,7 @@ jQuery.extend({
 	},
 
 	clean: function( elems, context, fragment, scripts ) {
-		var i, j, elem, tag, wrap, depth, div, hasBody, tbody, len, handleScript, jsTags,
+		var i, j, elem, tag, wrap, depth, div, hasBody, tbody, handleScript, jsTags,
 			safe = context === document && safeFragment,
 			ret = [];
 
diff --git a/src/serialize.js b/src/serialize.js
index 212640cf1..b81c15212 100644
--- a/src/serialize.js
+++ b/src/serialize.js
@@ -23,7 +23,7 @@ jQuery.fn.extend({
 			return val == null ?
 				null :
 				jQuery.isArray( val ) ?
-					jQuery.map( val, function( val, i ){
+					jQuery.map( val, function( val ){
 						return { name: elem.name, value: val.replace( rCRLF, "\r\n" ) };
 					}) :
 					{ name: elem.name, value: val.replace( rCRLF, "\r\n" ) };
diff --git a/src/traversing.js b/src/traversing.js
index 365bb2947..3e04bf4fb 100644
--- a/src/traversing.js
+++ b/src/traversing.js
@@ -271,7 +271,7 @@ function winnow( elements, qualifier, keep ) {
 		});
 
 	} else if ( qualifier.nodeType ) {
-		return jQuery.grep(elements, function( elem, i ) {
+		return jQuery.grep(elements, function( elem ) {
 			return ( elem === qualifier ) === keep;
 		});
 
@@ -287,7 +287,7 @@ function winnow( elements, qualifier, keep ) {
 		}
 	}
 
-	return jQuery.grep(elements, function( elem, i ) {
+	return jQuery.grep(elements, function( elem ) {
 		return ( jQuery.inArray( elem, qualifier ) >= 0 ) === keep;
 	});
 }