Skip to content

add multiple assignment support to java.util.Collection's Fixes #459#460

Merged
headius merged 1 commit intojruby:masterfrom
atambo:master
Mar 22, 2013
Merged

add multiple assignment support to java.util.Collection's Fixes #459#460
headius merged 1 commit intojruby:masterfrom
atambo:master

Conversation

@atambo
Copy link
Member

@atambo atambo commented Dec 22, 2012

No description provided.

@nirvdrum
Copy link
Contributor

Should this only work with java.util.List implementations? java.util.Set inherits from java.util.Collection, but Ruby doesn't exhibit the same multiple assignment behaviour with Sets, presumably because there is no ordering.

@atambo
Copy link
Member Author

atambo commented Dec 22, 2012

@nirvdrum, what about a Queue where there is ordering but it is not a List?

@atambo
Copy link
Member Author

atambo commented Dec 22, 2012

We'd also miss the SortedSet interface if we went List only. I feel like we should just do it for all collections since it will still work it just wont have any guarantees about order.

@nirvdrum
Copy link
Contributor

Ahh, more corner cases. I guess I don't have a strong issue with it being on java.util.Collection, other than it doesn't have 1:1 parity with Ruby's Set. Since #459 seemed to be about deviating behaviour, I figured I'd mention that this breaks in a different (probably less important) way.

headius added a commit that referenced this pull request Mar 22, 2013
add multiple assignment support to java.util.Collection's Fixes #459
@headius headius merged commit ff74833 into jruby:master Mar 22, 2013
@headius
Copy link
Member

headius commented Mar 22, 2013

Good fix, I like it. Merged.

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.

3 participants