Skip to content

(peazy) enumerator backports for 1.7#4563

Merged
kares merged 8 commits intojruby-1_7from
test-enumerator-backport-1_7
Apr 20, 2017
Merged

(peazy) enumerator backports for 1.7#4563
kares merged 8 commits intojruby-1_7from
test-enumerator-backport-1_7

Conversation

@kares
Copy link
Member

@kares kares commented Apr 19, 2017

~ cherry-picked from master, so that feed and some others are available

resolves #1571

@kares kares added this to the JRuby 1.7.27 milestone Apr 19, 2017
@enebo
Copy link
Member

enebo commented Apr 19, 2017

@kares thanks for backporting/picking back to 1.7. My only concern with this work is adding 2.x semantics but I can see (I only checked seek) was added in 1.9.x. So long as compat is cool bombs away!

@kares
Copy link
Member Author

kares commented Apr 19, 2017

@enebo thanks, I did check the docs. feed, peek_values and next_values are there in 1.9.3

@enebo
Copy link
Member

enebo commented Apr 19, 2017

@kares yeah I assumed you checked to see if the methods existed in 1.9. I was partially mentioning semantics of the methods themselves but I stated that perhaps in too general of a way.

@headius
Copy link
Member

headius commented Apr 19, 2017

👍 Looks good.

@kares
Copy link
Member Author

kares commented Apr 20, 2017

ok, semantics isn't 100% MRI compatible like on 9K and since this is simply a back-port it will do the same.
Enumerator#feed acts different but it has been decided to be fine for 9K and it hasn't been changing in Ruby 2.x so that is why I thought its better than nothing esp. since there's an old bug report asking for it for 1.7

@enebo
Copy link
Member

enebo commented Apr 20, 2017

@kares yeah I was just asking for due dilligence on whether there was some 2.0ish behavior making back to 1.9. So as long as you thought about it I definitely trust your judgement.

@kares
Copy link
Member Author

kares commented Apr 20, 2017

@enebo great thanks. I am grateful as I am never 100% sure either on these. I'll squash and merge than.

@kares kares merged commit c4198ee into jruby-1_7 Apr 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants