diff options
author | Rex Kerr <ichoran@gmail.com> | 2015-01-30 18:27:52 -0800 |
---|---|---|
committer | Rex Kerr <ichoran@gmail.com> | 2015-02-13 16:00:45 -0800 |
commit | c9930bfaf28093fe419de3cf034b4bf7888b3818 (patch) | |
tree | 4437a12d61d77006be8aa9a4651081ab53589f7d /src/library | |
parent | da49d9a00ec373a0e7f2ffe946a897a65c9b0741 (diff) | |
download | scala-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/library')
-rw-r--r-- | src/library/scala/collection/mutable/BitSet.scala | 6 |
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 } |