Skip to content

Iterator#sameElements has a different signature between 2.12 and 2.13 #75

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
Closed
julienrf opened this issue Jun 29, 2018 · 0 comments · Fixed by #78
Closed

Iterator#sameElements has a different signature between 2.12 and 2.13 #75

julienrf opened this issue Jun 29, 2018 · 0 comments · Fixed by #78
Milestone
Week 27

Comments

@julienrf
Copy link
Contributor

iterable.sameElements(iterable) compiles in 2.12, but in 2.13 the recommended way to write this is to use iterable.iterator.sameElements(iterable). However, that form does not cross-compile with 2.12, because sameElements take an Iterator in 2.12, whereas it takes an IterableOnce in 2.13. We can produce 2.12-compatible code by producing iterable.iterator.sameElements(iterable.iterator) instead.
@julienrf julienrf added this to the Week 27 milestone Jun 29, 2018
@julienrf julienrf changed the title Iterator#sameElements takes another Iterator as parameter Iterator#sameElements has a different signature between 2.12 and 2.13 Jun 29, 2018
@MasseGuillaume MasseGuillaume mentioned this issue Jul 2, 2018
Merged
martijnhoekstra pushed a commit to martijnhoekstra/scala-collection-compat that referenced this issue Nov 9, 2022
@julienrf
Merge pull request scala#75 from linasm/add-inplace-bitset-shifts-to-…
b8711fe 
…readme

Add in-place BitSet shift ops to README
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Week 27
Development

Successfully merging a pull request may close this issue.

Add more rules to CrossCompile MasseGuillaume/scala-collection-compat
1 participant
@julienrf