X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=src%2Fcore.js;h=3ba2a859d9682d85a0b499d60804ff0c81469dab;hb=ac00fe5bbb2a95fdb747f76fd9dd7c58ba6a531c;hp=b82133635fb2d19f0a34eec81f7daeac4e8ae6d0;hpb=b30af34f28074b491929445f5aad3d62c63e772f;p=jquery.git diff --git a/src/core.js b/src/core.js index b821336..3ba2a85 100644 --- a/src/core.js +++ b/src/core.js @@ -132,9 +132,9 @@ jQuery.fn = jQuery.prototype = { this.context = selector.context; } - return this.setArray(jQuery.isArray( selector ) ? - selector : - jQuery.makeArray(selector)); + return jQuery.isArray( selector ) ? + this.setArray( selector ) : + jQuery.makeArray( selector, this ); }, // Start with an empty selector @@ -419,19 +419,16 @@ jQuery.extend({ return (text || "").replace( rtrim, "" ); }, - makeArray: function( array ) { - var ret = [], i; + // results is for internal usage only + makeArray: function( array, results ) { + var ret = results || []; if ( array != null ) { - i = array.length; - // The window, strings (and functions) also have 'length' - if ( i == null || typeof array === "string" || jQuery.isFunction(array) || array.setInterval ) { - ret[0] = array; + if ( array.length == null || typeof array === "string" || jQuery.isFunction(array) || array.setInterval ) { + push.call( ret, array ); } else { - while ( i ) { - ret[--i] = array[i]; - } + jQuery.merge( ret, array ); } } @@ -453,12 +450,21 @@ jQuery.extend({ }, merge: function( first, second ) { - // We have to loop this way because IE & Opera overwrite the length - // expando of getElementsByTagName - var i = 0, elem, pos = first.length; + var pos, i = second.length; - while ( (elem = second[ i++ ]) != null ) { - first[ pos++ ] = elem; + // We have to get length this way when IE & Opera overwrite the length + // expando of getElementsByTagName + if ( i && i.nodeType ) { + for ( i = 0; second[i]; ++i ) {} + } + + pos = i + first.length; + + // Correct length for non Arrays + first.length = pos; + + while ( i ) { + first[ --pos ] = second[ --i ]; } return first;