X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=src%2Fselector%2Fselector.js;h=6a5760a70ab20b65e25e980dcb69cbb3a3a35fe7;hb=34355cd6986d5939dc711c531263cb561cba5f24;hp=8cff87fddc15311a5f5c6e64ebab69dcc7458825;hpb=866187fff66ef50b9c4b4c47f332f742c7546cc5;p=jquery.git diff --git a/src/selector/selector.js b/src/selector/selector.js index 8cff87f..6a5760a 100644 --- a/src/selector/selector.js +++ b/src/selector/selector.js @@ -153,10 +153,11 @@ jQuery.extend({ if ( m ) { // Perform our own iteration and filter - for ( var i = 0, rl = ret.length; i < rl; i++ ) - for ( var c = ret[i].firstChild; c; c = c.nextSibling ) + jQuery.each( ret, function(){ + for ( var c = this.firstChild; c; c = c.nextSibling ) if ( c.nodeType == 1 && ( c.nodeName == m[1].toUpperCase() || m[1] == "*" ) ) r.push( c ); + }); ret = r; t = jQuery.trim( t.replace( re, "" ) ); @@ -235,20 +236,20 @@ jQuery.extend({ // We need to find all descendant elements, it is more // efficient to use getAll() when we are already further down // the tree - we try to recognize that here - for ( var i = 0, rl = ret.length; i < rl; i++ ) { + jQuery.each( ret, function(){ // Grab the tag name being searched for var tag = m[1] != "" || m[0] == "" ? "*" : m[2]; // Handle IE7 being really dumb about s - if ( ret[i].nodeName.toUpperCase() == "OBJECT" && tag == "*" ) + if ( this.nodeName.toUpperCase() == "OBJECT" && tag == "*" ) tag = "param"; jQuery.merge( r, m[1] != "" && ret.length != 1 ? - jQuery.getAll( ret[i], [], m[1], m[2], rec ) : - ret[i].getElementsByTagName( tag ) + jQuery.getAll( this, [], m[1], m[2], rec ) : + this.getElementsByTagName( tag ) ); - } + }); // It's faster to filter by class and be done with it if ( m[1] == "." && ret.length == 1 ) @@ -263,11 +264,12 @@ jQuery.extend({ r = []; // Then try to find the element with the ID - for ( var i = 0, tl = tmp.length; i < tl; i++ ) - if ( tmp[i].getAttribute("id") == m[2] ) { - r = [ tmp[i] ]; - break; + jQuery.each( tmp, function(){ + if ( this.getAttribute("id") == m[2] ) { + r = [ this ]; + return false; } + }); } ret = r; @@ -300,14 +302,13 @@ jQuery.extend({ // Look for common filter expressions while ( t && /^[a-z[({<*:.#]/i.test(t) ) { - var p = jQuery.parse; + var p = jQuery.parse, m; - for ( var i = 0, pl = p.length; i < pl; i++ ) { + jQuery.each( p, function(i,re){ // Look for, and replace, string-like sequences // and finally build a regexp out of it - var re = p[i]; - var m = re.exec( t ); + m = re.exec( t ); if ( m ) { // Remove what we just matched @@ -317,9 +318,9 @@ jQuery.extend({ if ( jQuery.expr[ m[1] ]._resort ) m = jQuery.expr[ m[1] ]._resort( m ); - break; + return false; } - } + }); // :not() is a special case that can be optimized by // keeping it out of the expression list