Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enumerable#each_with_index is broken again #4724

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

Enumerable#each_with_index is broken again #4724

enebo opened this issue Jul 26, 2017 · 2 comments
Labels
core regression
Milestone
JRuby 9.1.13.0

Comments

@enebo
Copy link
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 enebo added this to the JRuby 9.2.0.0 milestone Jul 26, 2017
@enebo enebo mentioned this issue Jul 26, 2017
Closed
@enebo enebo modified the milestones: JRuby 9.1.13.0, JRuby 9.2.0.0 Aug 16, 2017
@enebo
Copy link
Member Author

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 as completed Aug 16, 2017
@enebo
Copy link
Member Author

enebo commented Aug 16, 2017

Added regression 04dd4a7 spec.

@koic koic mentioned this issue Sep 7, 2017
Merged
@mascolim mascolim mentioned this issue Oct 19, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core regression
Projects
None yet
Milestone
JRuby 9.1.13.0
Development

No branches or pull requests
1 participant
@enebo