diff options
author | Jason Zaugg <jzaugg@gmail.com> | 2013-09-09 13:48:11 -0700 |
---|---|---|
committer | Jason Zaugg <jzaugg@gmail.com> | 2013-09-09 13:48:11 -0700 |
commit | 6c431e66f591acdb104f09adaa935dc85e71a469 (patch) | |
tree | 576240e9973acb6576cd1631b3fa97e8581e46ff /src | |
parent | d1cff2078c4687ee1770bc9493791d2053b51a92 (diff) | |
parent | fe9a3e9c5ef2d2de5fa62a14fa4a613a573e5e5f (diff) | |
download | scala-6c431e66f591acdb104f09adaa935dc85e71a469.tar.gz scala-6c431e66f591acdb104f09adaa935dc85e71a469.tar.bz2 scala-6c431e66f591acdb104f09adaa935dc85e71a469.zip |
Merge pull request #2866 from retronym/ticket/7269
SI-7269 Rework MapLike#retains to account for desugaring change
Diffstat (limited to 'src')
-rw-r--r-- | src/library/scala/collection/mutable/MapLike.scala | 4 | ||||
-rw-r--r-- | src/library/scala/collection/mutable/SetLike.scala | 4 |
2 files changed, 5 insertions, 3 deletions
diff --git a/src/library/scala/collection/mutable/MapLike.scala b/src/library/scala/collection/mutable/MapLike.scala index a53aa3b76a..42e5a0a4c4 100644 --- a/src/library/scala/collection/mutable/MapLike.scala +++ b/src/library/scala/collection/mutable/MapLike.scala @@ -209,8 +209,8 @@ trait MapLike[A, B, +This <: MapLike[A, B, This] with Map[A, B]] * @param p The test predicate */ def retain(p: (A, B) => Boolean): this.type = { - for ((k, v) <- this.seq ; if !p(k, v)) - this -= k + for ((k, v) <- this.toList) // SI-7269 toList avoids ConcurrentModificationException + if (!p(k, v)) this -= k this } diff --git a/src/library/scala/collection/mutable/SetLike.scala b/src/library/scala/collection/mutable/SetLike.scala index 01f87447ae..71da4c89dc 100644 --- a/src/library/scala/collection/mutable/SetLike.scala +++ b/src/library/scala/collection/mutable/SetLike.scala @@ -120,7 +120,9 @@ trait SetLike[A, +This <: SetLike[A, This] with Set[A]] * which `p` returns `true` are retained in the set; all others * are removed. */ - def retain(p: A => Boolean): Unit = for (elem <- this.toList) if (!p(elem)) this -= elem + def retain(p: A => Boolean): Unit = + for (elem <- this.toList) // SI-7269 toList avoids ConcurrentModificationException + if (!p(elem)) this -= elem /** Removes all elements from the set. After this operation is completed, * the set will be empty. |