bjoernager/rust
bjoernager/rust
1
Fork 0
Code Issues Pull requests Activity

Fix cloning from a BitSet with a different domain size

Browse source 
The previous implementation incorrectly assumed that the
number of words in a bit set is equal to the domain size.

The new implementation delegates to `Vec::clone_from` which
is specialized for `Copy` elements.
This commit is contained in:
Tomasz Miąsko 2022-07-08 00:00:00 +00:00
parent eba361ae36
commit 9139a63b25
2 changed files with 19 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

17
compiler/rustc_index/src/bit_set/tests.rs
View file

@ -40,6 +40,23 @@ fn bitset_iter_works_2() {
assert_eq!(bitset.iter().collect::<Vec<_>>(), [0, 127, 191, 255, 319]);
}
#[test]
fn bitset_clone_from() {
let mut a: BitSet<usize> = BitSet::new_empty(10);
a.insert(4);
a.insert(7);
a.insert(9);
let mut b = BitSet::new_empty(2);
b.clone_from(&a);
assert_eq!(b.domain_size(), 10);
assert_eq!(b.iter().collect::<Vec<_>>(), [4, 7, 9]);
b.clone_from(&BitSet::new_empty(40));
assert_eq!(b.domain_size(), 40);
assert_eq!(b.iter().collect::<Vec<_>>(), []);
}
#[test]
fn union_two_sets() {
let mut set1: BitSet<usize> = BitSet::new_empty(65);