New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enumerable#each_with_index is broken again #4724

Closed
enebo opened this Issue Jul 26, 2017 · 2 comments

Comments

Projects
None yet
1 participant
@enebo
Member

enebo commented Jul 26, 2017

jruby -e '%i(foo bar baz).each_with_index {|v| puts v.inspect }

Prints out [:foo, 0], ... instead of :foo.

We broke this previous to 9.1.10.0 and then regressed again in .11/.12

Moved this over from #4610 so we can keep historical accuracy.

@enebo

This comment has been minimized.

Show comment
Hide comment
@enebo

enebo Aug 16, 2017

Member

Fixed in commit d597e1a on master and cherry picked back to jruby-9.1.13.0 branch. Not sure what happened here. I am guessing I only reverted on 9.1.10.0 branch? Will now see if a spec was only committed to that branch :|

Member

enebo commented Aug 16, 2017

Fixed in commit d597e1a on master and cherry picked back to jruby-9.1.13.0 branch. Not sure what happened here. I am guessing I only reverted on 9.1.10.0 branch? Will now see if a spec was only committed to that branch :|

@enebo enebo closed this Aug 16, 2017

@enebo

This comment has been minimized.

Show comment
Hide comment
@enebo

enebo Aug 16, 2017

Member

Added regression 04dd4a7 spec.

Member

enebo commented Aug 16, 2017

Added regression 04dd4a7 spec.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment