Skip to content

Loading…

Update casbah-commons/src/main/scala/MongoDBList.scala #51

Closed
wants to merge 2 commits into from

2 participants

@negator

The MongoDBList.concat(..) function, seems to force the use of a mutable Traversable object, due to the
'import scala.collection.mutable._' statement on line 27. 

This function should accept a scala.collection.Traversable. 

negator added some commits
@negator negator Update casbah-commons/src/main/scala/MongoDBList.scala
The MongoDBList.concat(..) function, seems to force the use of a mutable Traversable object, due to the
'import scala.collection.mutable._' statement on line 27. 

This function should accept a scala.collection.Traversable. 
5d9634f
@negator negator Update casbah-commons/src/main/scala/MongoDBList.scala 48f39a6
@rozza
mongodb member

Thanks for the code - this will be in 2.5.0

@rozza rozza closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 8, 2012
  1. @negator

    Update casbah-commons/src/main/scala/MongoDBList.scala

    negator committed
    The MongoDBList.concat(..) function, seems to force the use of a mutable Traversable object, due to the
    'import scala.collection.mutable._' statement on line 27. 
    
    This function should accept a scala.collection.Traversable. 
Commits on Oct 9, 2012
  1. @negator
Showing with 2 additions and 2 deletions.
  1. +2 −2 casbah-commons/src/main/scala/MongoDBList.scala
View
4 casbah-commons/src/main/scala/MongoDBList.scala
@@ -44,7 +44,7 @@ class MongoDBList(val underlying: BasicDBList = new BasicDBList) extends Seq[Any
this
}
- def insertAll(i: Int, elems: Traversable[Any]) = {
+ def insertAll(i: Int, elems: scala.collection.Traversable[Any]) = {
val ins = underlying.subList(0, i)
elems.foreach(x => ins.add(x.asInstanceOf[AnyRef]))
}
@@ -117,7 +117,7 @@ object MongoDBList {
b.result
}
- def concat[A](xss: Traversable[A]*): MongoDBList = {
+ def concat[A](xss: scala.collection.Traversable[A]*): MongoDBList = {
val b = newBuilder[A]
if (xss forall (_.isInstanceOf[IndexedSeq[_]]))
b.sizeHint(xss map (_.size) sum)
Something went wrong with that request. Please try again.