Skip to content
This repository was archived by the owner on Dec 22, 2021. It is now read-only.

Conversation

@julienrf
Copy link
Contributor

@julienrf julienrf commented Jan 25, 2018

When cross-compiling better-files I noticed a few things that were missing:

  • an overload of collect returning a Map in MapOps,
  • deprecated toIterator operation in IterableOnce.

@julienrf julienrf requested review from lrytz and szeiger January 25, 2018 12:55

@deprecated("Use Set.from(it) instead of it.toSet", "2.13.0")
@`inline` final def toSet[B >: A]: immutable.Set[B] = immutable.Set.from(it)
@`inline` def toSet[B >: A]: immutable.Set[B] = immutable.Set.from(it)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you un-finalize these methods? Note that @inline only works on final methods.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because the outer class is already final.

@julienrf julienrf merged commit 80bebef into scala:master Jan 29, 2018
@julienrf julienrf deleted the compatibility branch January 29, 2018 08:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants