From 7a467ed9360c2c55ea55a56e5767c9581e4f4f69 Mon Sep 17 00:00:00 2001 From: jeresig Date: Sat, 13 Feb 2010 01:45:09 -0500 Subject: [PATCH] Remove the need for the internal setArray method, it was only really used by pushStack anyway. Fixes #6003. --- src/core.js | 21 ++++++++------------- 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/src/core.js b/src/core.js index 34bab95..02d8269 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.) -- 1.7.10.4