From: Oleg Date: Mon, 7 Jan 2013 21:59:01 +0000 (+0400) Subject: Use while loop instead of for X-Git-Tag: 2.0.0b1~24 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=57d9dcd4a0c3a56160144b7f468fff6e419fed84;p=jquery.git Use while loop instead of for --- 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 ); }