From: Sindre Sorhus Date: Mon, 7 May 2012 17:59:11 +0000 (+0200) Subject: Remove unnecessary arguments from .nth(). Fixes #11720 X-Git-Tag: 1.8b1~157^2 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=4de7b5412e33331ca6f9f8a2d6e5e00dec7299f2;p=jquery.git Remove unnecessary arguments from .nth(). Fixes #11720 Also use postfix increment to make it a little clearer. --- diff --git a/src/traversing.js b/src/traversing.js index 2e45aade7..0b8a5f240 100644 --- a/src/traversing.js +++ b/src/traversing.js @@ -186,10 +186,10 @@ jQuery.each({ return jQuery.dir( elem, "parentNode", until ); }, next: function( elem ) { - return jQuery.nth( elem, 2, "nextSibling" ); + return jQuery.nth( elem, "nextSibling" ); }, prev: function( elem ) { - return jQuery.nth( elem, 2, "previousSibling" ); + return jQuery.nth( elem, "previousSibling" ); }, nextAll: function( elem ) { return jQuery.dir( elem, "nextSibling" ); @@ -260,12 +260,11 @@ jQuery.extend({ return matched; }, - nth: function( cur, result, dir, elem ) { - result = result || 1; + nth: function( cur, dir ) { var num = 0; for ( ; cur; cur = cur[dir] ) { - if ( cur.nodeType === 1 && ++num === result ) { + if ( cur.nodeType === 1 && num++ === 1 ) { break; } }