From: jeresig <jeresig@gmail.com>
Date: Sat, 13 Feb 2010 06:45:09 +0000 (-0500)
Subject: Remove the need for the internal setArray method, it was only really used by pushStac... 
X-Git-Tag: 1.4.2~19
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=7a467ed9360c2c55ea55a56e5767c9581e4f4f69;p=jquery.git

Remove the need for the internal setArray method, it was only really used by pushStack anyway. Fixes #6003.
---

diff --git a/src/core.js b/src/core.js
index 34bab952d..02d8269d2 100644
--- a/src/core.js
+++ b/src/core.js
@@ -197,7 +197,14 @@ jQuery.fn = jQuery.prototype = {
 	// (returning the new matched element set)
 	pushStack: function( elems, name, selector ) {
 		// Build a new jQuery matched element set
-		var ret = jQuery( elems || null );
+		var ret = jQuery();
+
+		if ( jQuery.isArray( elems ) ) {
+			push.apply( ret, elems );
+		
+		} else {
+			jQuery.merge( ret, elems );
+		}
 
 		// Add the old object onto the stack (as a reference)
 		ret.prevObject = this;
@@ -214,18 +221,6 @@ jQuery.fn = jQuery.prototype = {
 		return ret;
 	},
 
-	// Force the current matched set of elements to become
-	// the specified array of elements (destroying the stack in the process)
-	// You should use pushStack() in order to do this, but maintain the stack
-	setArray: function( elems ) {
-		// Resetting the length to 0, then using the native Array push
-		// is a super-fast way to populate an object with array-like properties
-		this.length = 0;
-		push.apply( this, elems );
-
-		return this;
-	},
-
 	// Execute a callback for every element in the matched set.
 	// (You can seed the arguments with an array of args, but this is
 	// only used internally.)