Skip to content

Commit fb48ae8

Browse files
committed
Make sure that when multiple variables are being declared that assignments are each done on their own line.
1 parent b5b3c73 commit fb48ae8

File tree

12 files changed

+98
-45
lines changed

12 files changed

+98
-45
lines changed

src/ajax.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -506,11 +506,12 @@ jQuery.extend({
506506
// Serialize an array of form elements or a set of
507507
// key/values into a query string
508508
param: function( a, traditional ) {
509-
var s = [], add = function( key, value ) {
510-
// If value is a function, invoke it and return its value
511-
value = jQuery.isFunction(value) ? value() : value;
512-
s[ s.length ] = encodeURIComponent(key) + "=" + encodeURIComponent(value);
513-
};
509+
var s = [],
510+
add = function( key, value ) {
511+
// If value is a function, invoke it and return its value
512+
value = jQuery.isFunction(value) ? value() : value;
513+
s[ s.length ] = encodeURIComponent(key) + "=" + encodeURIComponent(value);
514+
};
514515

515516
// Set traditional to true for jQuery <= 1.3.2 behavior.
516517
if ( traditional === undefined ) {

src/attributes.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,9 @@ jQuery.fn.extend({
5555
elem.className = value;
5656

5757
} else {
58-
var className = " " + elem.className + " ", setClass = elem.className;
58+
var className = " " + elem.className + " ",
59+
setClass = elem.className;
60+
5961
for ( var c = 0, cl = classNames.length; c < cl; c++ ) {
6062
if ( className.indexOf( " " + classNames[c] + " " ) < 0 ) {
6163
setClass += " " + classNames[c];
@@ -103,7 +105,8 @@ jQuery.fn.extend({
103105
},
104106

105107
toggleClass: function( value, stateVal ) {
106-
var type = typeof value, isBool = typeof stateVal === "boolean";
108+
var type = typeof value,
109+
isBool = typeof stateVal === "boolean";
107110

108111
if ( jQuery.isFunction( value ) ) {
109112
return this.each(function(i) {
@@ -115,7 +118,9 @@ jQuery.fn.extend({
115118
return this.each(function() {
116119
if ( type === "string" ) {
117120
// toggle individual class names
118-
var className, i = 0, self = jQuery(this),
121+
var className,
122+
i = 0,
123+
self = jQuery( this ),
119124
state = stateVal,
120125
classNames = value.split( rspaces );
121126

src/core.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -311,8 +311,11 @@ jQuery.fn = jQuery.prototype = {
311311
jQuery.fn.init.prototype = jQuery.fn;
312312

313313
jQuery.extend = jQuery.fn.extend = function() {
314-
// copy reference to target object
315-
var target = arguments[0] || {}, i = 1, length = arguments.length, deep = false, options, name, src, copy, copyIsArray, clone;
314+
var options, name, src, copy, copyIsArray, clone,
315+
target = arguments[0] || {},
316+
i = 1,
317+
length = arguments.length,
318+
deep = false;
316319

317320
// Handle a deep copy situation
318321
if ( typeof target === "boolean" ) {
@@ -678,7 +681,8 @@ jQuery.extend({
678681
},
679682

680683
merge: function( first, second ) {
681-
var i = first.length, j = 0;
684+
var i = first.length,
685+
j = 0;
682686

683687
if ( typeof second.length === "number" ) {
684688
for ( var l = second.length; j < l; j++ ) {

src/css.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,9 @@ if ( document.defaultView && document.defaultView.getComputedStyle ) {
258258

259259
if ( document.documentElement.currentStyle ) {
260260
currentStyle = function( elem, name ) {
261-
var left, rsLeft, ret = elem.currentStyle && elem.currentStyle[ name ], style = elem.style;
261+
var left, rsLeft,
262+
ret = elem.currentStyle && elem.currentStyle[ name ],
263+
style = elem.style;
262264

263265
// From the awesome hack by Dean Edwards
264266
// http://erik.eae.net/archives/2007/07/27/18.54.15/#comment-102291
@@ -312,7 +314,8 @@ function getWH( elem, name, extra ) {
312314

313315
if ( jQuery.expr && jQuery.expr.filters ) {
314316
jQuery.expr.filters.hidden = function( elem ) {
315-
var width = elem.offsetWidth, height = elem.offsetHeight;
317+
var width = elem.offsetWidth,
318+
height = elem.offsetHeight;
316319

317320
return (width === 0 && height === 0) || (!jQuery.support.reliableHiddenOffsets && (elem.style.display || jQuery.css( elem, "display" )) === "none");
318321
};

src/data.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,8 @@ jQuery.fn.extend({
177177

178178
} else {
179179
return this.each(function() {
180-
var $this = jQuery( this ), args = [ parts[0], value ];
180+
var $this = jQuery( this ),
181+
args = [ parts[0], value ];
181182

182183
$this.triggerHandler( "setData" + parts[1] + "!", args );
183184
jQuery.data( this, key, value );

src/dimensions.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,9 @@ jQuery.each([ "Height", "Width" ], function( i, name ) {
4949

5050
// Get or set width or height on the element
5151
} else if ( size === undefined ) {
52-
var orig = jQuery.css( elem, type ), ret = parseFloat( orig );
52+
var orig = jQuery.css( elem, type ),
53+
ret = parseFloat( orig );
54+
5355
return jQuery.isNaN( ret ) ? orig : ret;
5456

5557
// Set the width or height on the element (default to pixels if value is unitless)

src/effects.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -342,14 +342,16 @@ jQuery.fx.prototype = {
342342

343343
// Start an animation from one number to another
344344
custom: function( from, to, unit ) {
345+
var self = this,
346+
fx = jQuery.fx;
347+
345348
this.startTime = jQuery.now();
346349
this.start = from;
347350
this.end = to;
348351
this.unit = unit || this.unit || "px";
349352
this.now = this.start;
350353
this.pos = this.state = 0;
351354

352-
var self = this, fx = jQuery.fx;
353355
function t( gotoEnd ) {
354356
return self.step(gotoEnd);
355357
}
@@ -406,7 +408,9 @@ jQuery.fx.prototype = {
406408
if ( done ) {
407409
// Reset the overflow
408410
if ( this.options.overflow != null && !jQuery.support.shrinkWrapBlocks ) {
409-
var elem = this.elem, options = this.options;
411+
var elem = this.elem,
412+
options = this.options;
413+
410414
jQuery.each( [ "", "X", "Y" ], function (index, value) {
411415
elem.style[ "overflow" + value ] = options.overflow[index];
412416
} );

src/event.js

Lines changed: 19 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -378,8 +378,10 @@ jQuery.event = {
378378
jQuery.event.trigger( event, data, parent, true );
379379

380380
} else if ( !event.isDefaultPrevented() ) {
381-
var target = event.target, old, targetType = type.replace(rnamespaces, ""),
382-
isClick = jQuery.nodeName(target, "a") && targetType === "click",
381+
var old,
382+
target = event.target,
383+
targetType = type.replace( rnamespaces, "" ),
384+
isClick = jQuery.nodeName( target, "a" ) && targetType === "click",
383385
special = jQuery.event.special[ targetType ] || {};
384386

385387
if ( (!special._default || special._default.call( elem, event ) === false) &&
@@ -411,7 +413,9 @@ jQuery.event = {
411413
},
412414

413415
handle: function( event ) {
414-
var all, handlers, namespaces, namespace_sort = [], namespace_re, events, args = jQuery.makeArray( arguments );
416+
var all, handlers, namespaces, namespace_re, events,
417+
namespace_sort = [],
418+
args = jQuery.makeArray( arguments );
415419

416420
event = args[0] = jQuery.event.fix( event || window.event );
417421
event.currentTarget = this;
@@ -506,7 +510,9 @@ jQuery.event = {
506510

507511
// Calculate pageX/Y if missing and clientX/Y available
508512
if ( event.pageX == null && event.clientX != null ) {
509-
var doc = document.documentElement, body = document.body;
513+
var doc = document.documentElement,
514+
body = document.body;
515+
510516
event.pageX = event.clientX + (doc && doc.scrollLeft || body && body.scrollLeft || 0) - (doc && doc.clientLeft || body && body.clientLeft || 0);
511517
event.pageY = event.clientY + (doc && doc.scrollTop || body && body.scrollTop || 0) - (doc && doc.clientTop || body && body.clientTop || 0);
512518
}
@@ -712,7 +718,8 @@ if ( !jQuery.support.submitBubbles ) {
712718
setup: function( data, namespaces ) {
713719
if ( this.nodeName.toLowerCase() !== "form" ) {
714720
jQuery.event.add(this, "click.specialSubmit", function( e ) {
715-
var elem = e.target, type = elem.type;
721+
var elem = e.target,
722+
type = elem.type;
716723

717724
if ( (type === "submit" || type === "image") && jQuery( elem ).closest("form").length ) {
718725
e.liveFired = undefined;
@@ -721,7 +728,8 @@ if ( !jQuery.support.submitBubbles ) {
721728
});
722729

723730
jQuery.event.add(this, "keypress.specialSubmit", function( e ) {
724-
var elem = e.target, type = elem.type;
731+
var elem = e.target,
732+
type = elem.type;
725733

726734
if ( (type === "text" || type === "password") && jQuery( elem ).closest("form").length && e.keyCode === 13 ) {
727735
e.liveFired = undefined;
@@ -962,7 +970,8 @@ jQuery.fn.extend({
962970

963971
toggle: function( fn ) {
964972
// Save reference to arguments for access in closure
965-
var args = arguments, i = 1;
973+
var args = arguments,
974+
i = 1;
966975

967976
// link all the functions, so any of them can unbind this click handler
968977
while ( i < args.length ) {
@@ -1057,8 +1066,9 @@ jQuery.each(["live", "die"], function( i, name ) {
10571066
});
10581067

10591068
function liveHandler( event ) {
1060-
var stop, maxLevel, elems = [], selectors = [],
1061-
related, match, handleObj, elem, j, i, l, data, close, namespace, ret,
1069+
var stop, maxLevel, related, match, handleObj, elem, j, i, l, data, close, namespace, ret,
1070+
elems = [],
1071+
selectors = [],
10621072
events = jQuery.data( this, this.nodeType ? "events" : "__events__" );
10631073

10641074
if ( typeof events === "function" ) {

src/manipulation.js

Lines changed: 21 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@ jQuery.fn.extend({
3434
text: function( text ) {
3535
if ( jQuery.isFunction(text) ) {
3636
return this.each(function(i) {
37-
var self = jQuery(this);
37+
var self = jQuery( this );
38+
3839
self.text( text.call(this, i, self.text()) );
3940
});
4041
}
@@ -83,7 +84,8 @@ jQuery.fn.extend({
8384
}
8485

8586
return this.each(function() {
86-
var self = jQuery( this ), contents = self.contents();
87+
var self = jQuery( this ),
88+
contents = self.contents();
8789

8890
if ( contents.length ) {
8991
contents.wrapAll( html );
@@ -194,7 +196,9 @@ jQuery.fn.extend({
194196
// attributes in IE that are actually only stored
195197
// as properties will not be copied (such as the
196198
// the name attribute on an input).
197-
var html = this.outerHTML, ownerDocument = this.ownerDocument;
199+
var html = this.outerHTML,
200+
ownerDocument = this.ownerDocument;
201+
198202
if ( !html ) {
199203
var div = ownerDocument.createElement("div");
200204
div.appendChild( this.cloneNode(true) );
@@ -249,7 +253,8 @@ jQuery.fn.extend({
249253

250254
} else if ( jQuery.isFunction( value ) ) {
251255
this.each(function(i){
252-
var self = jQuery(this);
256+
var self = jQuery( this );
257+
253258
self.html( value.call(this, i, self.html()) );
254259
});
255260

@@ -272,13 +277,14 @@ jQuery.fn.extend({
272277
}
273278

274279
if ( typeof value !== "string" ) {
275-
value = jQuery(value).detach();
280+
value = jQuery( value ).detach();
276281
}
277282

278283
return this.each(function() {
279-
var next = this.nextSibling, parent = this.parentNode;
284+
var next = this.nextSibling,
285+
parent = this.parentNode;
280286

281-
jQuery(this).remove();
287+
jQuery( this ).remove();
282288

283289
if ( next ) {
284290
jQuery(next).before( value );
@@ -296,7 +302,9 @@ jQuery.fn.extend({
296302
},
297303

298304
domManip: function( args, table, callback ) {
299-
var results, first, value = args[0], scripts = [], fragment, parent;
305+
var results, first, fragment, parent,
306+
value = args[0],
307+
scripts = [];
300308

301309
// We can't cloneNode fragments that contain checked, in WebKit
302310
if ( !jQuery.support.checkClone && arguments.length === 3 && typeof value === "string" && rchecked.test( value ) ) {
@@ -371,7 +379,9 @@ function cloneCopyEvent(orig, ret) {
371379
return;
372380
}
373381

374-
var oldData = jQuery.data( orig[i++] ), curData = jQuery.data( this, oldData ), events = oldData && oldData.events;
382+
var oldData = jQuery.data( orig[i++] ),
383+
curData = jQuery.data( this, oldData ),
384+
events = oldData && oldData.events;
375385

376386
if ( events ) {
377387
delete curData.handle;
@@ -428,7 +438,8 @@ jQuery.each({
428438
replaceAll: "replaceWith"
429439
}, function( name, original ) {
430440
jQuery.fn[ name ] = function( selector ) {
431-
var ret = [], insert = jQuery( selector ),
441+
var ret = [],
442+
insert = jQuery( selector ),
432443
parent = this.length === 1 && this[0].parentNode;
433444

434445
if ( parent && parent.nodeType === 11 && parent.childNodes.length === 1 && insert.length === 1 ) {

src/offset.js

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,16 @@ if ( "getBoundingClientRect" in document.documentElement ) {
6565

6666
jQuery.offset.initialize();
6767

68-
var offsetParent = elem.offsetParent, prevOffsetParent = elem,
69-
doc = elem.ownerDocument, computedStyle, docElem = doc.documentElement,
70-
body = doc.body, defaultView = doc.defaultView,
68+
var computedStyle,
69+
offsetParent = elem.offsetParent,
70+
prevOffsetParent = elem,
71+
doc = elem.ownerDocument,
72+
docElem = doc.documentElement,
73+
body = doc.body,
74+
defaultView = doc.defaultView,
7175
prevComputedStyle = defaultView ? defaultView.getComputedStyle( elem, null ) : elem.currentStyle,
72-
top = elem.offsetTop, left = elem.offsetLeft;
76+
top = elem.offsetTop,
77+
left = elem.offsetLeft;
7378

7479
while ( (elem = elem.parentNode) && elem !== body && elem !== docElem ) {
7580
if ( jQuery.offset.supportsFixedPosition && prevComputedStyle.position === "fixed" ) {
@@ -151,7 +156,8 @@ jQuery.offset = {
151156
},
152157

153158
bodyOffset: function( body ) {
154-
var top = body.offsetTop, left = body.offsetLeft;
159+
var top = body.offsetTop,
160+
left = body.offsetLeft;
155161

156162
jQuery.offset.initialize();
157163

0 commit comments

Comments
 (0)