deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

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

Browse source
This commit is contained in:
jeresig 2010-02-13 01:45:09 -05:00
parent 8404ad6070
commit 7a467ed936
1 changed files with 8 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
src/core.js
View file

@ -197,7 +197,14 @@ jQuery.fn = jQuery.prototype = {
// (returning the new matched element set) // (returning the new matched element set)
pushStack: function( elems, name, selector ) { pushStack: function( elems, name, selector ) {
// Build a new jQuery matched element set // 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) // Add the old object onto the stack (as a reference)
ret.prevObject = this; ret.prevObject = this;
@ -214,18 +221,6 @@ jQuery.fn = jQuery.prototype = {
return ret; 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. // Execute a callback for every element in the matched set.
// (You can seed the arguments with an array of args, but this is // (You can seed the arguments with an array of args, but this is
// only used internally.) // only used internally.)