Hacker Newsnew | comments | ask | jobs | submitlogin
crescentfresh 1085 days ago | link | parent

Simple reproduce case:

  <script src="http://code.jquery.com/jquery-1.6.min.js"></script>
The bug is in this block: https://github.com/jquery/jquery/blob/1.6/src/manipulation.j...

Note the inner loop is clobbering the outer loop's "i" variable.

jeresig 1085 days ago | link

Oof! That's no good, at all. We'll fix this right away. Thanks for the reduction. Just filed it here: http://bugs.jquery.com/ticket/9072


jeresig 1085 days ago | link

The issue has been fixed and integrated into the current build of jQuery: http://code.jquery.com/jquery-git.js

We're going to have a 1.6.1 release within the week. Thank you for your bug report!


DTrejo 1085 days ago | link

He gets a reward!



wyday 1085 days ago | link

Fantastic find, crescentfresh! Changing "i" to "j" in the inner loop fixes the problem on IE6.

Now we just have to wait for jQuery 1.6.1.


rwaldron 1085 days ago | link

In the future, please post any jQuery bugs you happen to find in the jQuery Bug Tracker http://bugs.jquery.com/

This helps us to help you! And remember, the future in San Dimas is excellent... so be excellent to each other.


Lists | RSS | Bookmarklet | Guidelines | FAQ | DMCA | News News | Feature Requests | Bugs | Y Combinator | Apply | Library