summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorRex Kerr <ichoran@gmail.com>2015-01-30 18:27:52 -0800
committerRex Kerr <ichoran@gmail.com>2015-02-13 16:00:45 -0800
commitc9930bfaf28093fe419de3cf034b4bf7888b3818 (patch)
tree4437a12d61d77006be8aa9a4651081ab53589f7d /src
parentda49d9a00ec373a0e7f2ffe946a897a65c9b0741 (diff)
downloadscala-c9930bfaf28093fe419de3cf034b4bf7888b3818.tar.gz
scala-c9930bfaf28093fe419de3cf034b4bf7888b3818.tar.bz2
scala-c9930bfaf28093fe419de3cf034b4bf7888b3818.zip
SI-8917 collection.mutable.BitSet's &= operator doesn't clear end
Made &= run to the end of its own bitset, ignoring the size of what it's &-ing with (which is exactly what you want for &=).
Diffstat (limited to 'src')
-rw-r--r--src/library/scala/collection/mutable/BitSet.scala6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/library/scala/collection/mutable/BitSet.scala b/src/library/scala/collection/mutable/BitSet.scala
index 78150b5e88..e92d48cfeb 100644
--- a/src/library/scala/collection/mutable/BitSet.scala
+++ b/src/library/scala/collection/mutable/BitSet.scala
@@ -121,8 +121,10 @@ class BitSet(protected final var elems: Array[Long]) extends AbstractSet[Int]
* @return the bitset itself.
*/
def &= (other: BitSet): this.type = {
- ensureCapacity(other.nwords - 1)
- for (i <- 0 until other.nwords)
+ // Different from other operations: no need to ensure capacity because
+ // anything beyond the capacity is 0. Since we use other.word which is 0
+ // off the end, we also don't need to make sure we stay in bounds there.
+ for (i <- 0 until nwords)
elems(i) = elems(i) & other.word(i)
this
}