diff options
author | Josh Suereth <Joshua.Suereth@gmail.com> | 2012-09-12 12:58:10 -0700 |
---|---|---|
committer | Josh Suereth <Joshua.Suereth@gmail.com> | 2012-09-12 12:58:10 -0700 |
commit | c168c69e8614f701b6c89e01bdd38a54e154e9cd (patch) | |
tree | 0ab8b20979acc392ff4a5bff77b0321d30796ea3 /src | |
parent | ea242102060286cd3921ce552142435960352712 (diff) | |
parent | 674e0f80016dd70cd04c533daceeaa12161dae17 (diff) | |
download | scala-c168c69e8614f701b6c89e01bdd38a54e154e9cd.tar.gz scala-c168c69e8614f701b6c89e01bdd38a54e154e9cd.tar.bz2 scala-c168c69e8614f701b6c89e01bdd38a54e154e9cd.zip |
Merge pull request #1247 from jsuereth/fix/SI-6114
Fix t6114 - ++ on JList wrapper modifies underlying collection.
Diffstat (limited to 'src')
-rw-r--r-- | src/library/scala/collection/convert/Wrappers.scala | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/library/scala/collection/convert/Wrappers.scala b/src/library/scala/collection/convert/Wrappers.scala index 75707b69b0..7c97972c48 100644 --- a/src/library/scala/collection/convert/Wrappers.scala +++ b/src/library/scala/collection/convert/Wrappers.scala @@ -96,6 +96,9 @@ private[collection] trait Wrappers { def remove(i: Int) = underlying.remove(i) def clear() = underlying.clear() def result = this + // Note: Clone cannot just call underlying.clone because in Java, only specific collections + // expose clone methods. Generically, they're protected. + override def clone(): JListWrapper[A] = JListWrapper(new ju.ArrayList[A](underlying)) } class SetWrapper[A](underlying: Set[A]) extends ju.AbstractSet[A] { @@ -149,6 +152,10 @@ private[collection] trait Wrappers { override def clear() = underlying.clear() override def empty = JSetWrapper(new ju.HashSet[A]) + // Note: Clone cannot just call underlying.clone because in Java, only specific collections + // expose clone methods. Generically, they're protected. + override def clone() = + new JSetWrapper[A](new ju.LinkedHashSet[A](underlying)) } class MapWrapper[A, B](underlying: Map[A, B]) extends ju.AbstractMap[A, B] { self => |