From 57d9dcd4a0c3a56160144b7f468fff6e419fed84 Mon Sep 17 00:00:00 2001 From: Oleg Date: Tue, 8 Jan 2013 01:59:01 +0400 Subject: [PATCH] Use while loop instead of for --- src/manipulation.js | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/src/manipulation.js b/src/manipulation.js index 3ddd56c3a..c85caa060 100644 --- a/src/manipulation.js +++ b/src/manipulation.js @@ -422,16 +422,12 @@ jQuery.extend({ }, buildFragment: function( elems, context, scripts, selection ) { - var elem, tmp, tag, wrap, j, ll, contains, - fragment = context.createDocumentFragment(), + var elem, tmp, tag, wrap, contains, j, i = 0, - l = elems.length, + fragment = context.createDocumentFragment(), nodes = []; - for ( ; i < l; i++ ) { - elem = elems[ i ]; - - if ( elem || elem === 0 ) { + while ( ( elem = elems[ i++ ] ) || elem === 0 ) { // Add nodes directly if ( jQuery.type( elem ) === "object" ) { @@ -465,14 +461,13 @@ jQuery.extend({ // Support: Webkit, IE tmp.textContent = ""; } - } } // Remove wrapper from fragment fragment.textContent = ""; - for ( i = 0, l = nodes.length; i < l; i++ ) { - elem = nodes[ i ]; + i = 0; + while ( (elem = nodes[ i++ ]) ) { contains = jQuery.contains( elem.ownerDocument, elem ); // #4087 - If origin and destination elements are the same, and this is @@ -491,9 +486,9 @@ jQuery.extend({ // Capture executables if ( scripts ) { - for ( j = 0, ll = tmp.length; j < ll; j++ ) { - elem = tmp[ j ]; + j = 0; + while ( (elem = tmp[ j++ ]) ) { if ( rscriptType.test( elem.type || "" ) ) { core_push.call( scripts, elem ); } -- 2.39.5