Commit 24e96ba2 authored by jeresig's avatar jeresig

Re-worked the closest no-duplicate-selector code again to be more efficient.

parent 525bb27b
...@@ -56,29 +56,26 @@ jQuery.fn.extend({ ...@@ -56,29 +56,26 @@ jQuery.fn.extend({
closest: function( selectors, context ) { closest: function( selectors, context ) {
if ( jQuery.isArray( selectors ) ) { if ( jQuery.isArray( selectors ) ) {
var ret = [], cur = this[0], match, selector, done; var ret = [], cur = this[0], match, matches = {}, selector;
if ( cur && selectors.length ) { if ( cur && selectors.length ) {
for ( var i = 0, l = selectors.length; i < l; i++ ) { for ( var i = 0, l = selectors.length; i < l; i++ ) {
selectors[i] = jQuery.expr.match.POS.test( selector ) ? selector = selectors[i];
jQuery( selector, context || this.context ) :
selectors[i]; if ( !matches[selector] ) {
matches[selector] = jQuery.expr.match.POS.test( selector ) ?
jQuery( selector, context || this.context ) :
selector;
}
} }
while ( cur && cur.ownerDocument && cur !== context ) { while ( cur && cur.ownerDocument && cur !== context ) {
for ( var i = 0; i < selectors.length; i++ ) { for ( selector in matches ) {
match = selectors[i]; match = matches[selector];
selector = match.selector || match;
// Get rid of duplicate selectors
if ( selector === done ) {
selectors.splice(i--, 1);
// See if we have a match if ( match.jquery ? match.index(cur) > -1 : jQuery(cur).is(match) ) {
} else if ( match.jquery ? match.index(cur) > -1 : jQuery(cur).is(match) ) {
ret.push({ selector: selector, elem: cur }); ret.push({ selector: selector, elem: cur });
selectors.splice(i--, 1); delete matches[selector];
done = selector;
} }
} }
cur = cur.parentNode; cur = cur.parentNode;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment