search

scala / collection-strawman

Implementation of the new Scala 2.13 Collections
Apache License 2.0
200 stars 72 forks source link

Re-add (deprecated?) collection.Seq.updated? #549

Closed lrytz closed 6 years ago

lrytz commented 6 years ago

Can work around with collection.Seq(1).patch(0, List(2), 1)...

Once fixed, revert fix TODO-newColl in test/files/run/t5328.scala.

szeiger commented 6 years ago

This is similar to methods like + that build a new collection but could easily mistaken for in-place updates and are therefore only defined (and not deprecated) for immutable collections. We should consider deprecating collection.Seq.updated as well.

julienrf commented 6 years ago

@szeiger We eventually pulled the +: and :+ operations of collection.immutable.Seq to collection.Seq, without deprecating them.