SynchronizedQueue doesn't synchronize foreach (and many other methods)

[scabug]

val queue = new SynchronizedQueue[(A,B)]

for( (a,_) <- queue ){
	// ...
}


java.util.NoSuchElementException
	at scala.collection.mutable.LinkedListLike$class.head(LinkedListLike.scala:69)
	at scala.collection.mutable.LinkedList.head(LinkedList.scala:41)
	at scala.collection.mutable.MutableList.head(MutableList.scala:51)
	at scala.collection.LinearSeqOptimized$class.foreach(LinearSeqOptimized.scala:59)
	at scala.collection.mutable.MutableList.foreach(MutableList.scala:28)
	at scala.collection.TraversableLike$class.filter(TraversableLike.scala:212)
	at scala.collection.mutable.MutableList.filter(MutableList.scala:28)
	at line 3 in the code example

the queue is also accessed by other Actors.

[scabug]

Imported From: https://issues.scala-lang.org/browse/SI-4916?orig=1
Reporter: Arne Döring (krux02)
Affected Versions: 2.9.0-1
See #4290
Duplicates #4290

[scabug]

@retronym said (edited on May 12, 2012 12:41:13 PM UTC):
This class of bug seems almost guaranteed in all the SynchronizedXxx collection proxies -- there are no checks in place to ensure every method is overridden with synchronized wrapper.

An admission of this in a TODO comment: https://github.com/scala/scala/blob/master/src/library/scala/collection/mutable/SynchronizedMap.scala#L57

Whacking a solitary mole:

#4326

[scabug]

@retronym said:
Merging with #4290