Try to find empty subbitmaps in treemap

RoaringBitmap · Feb 13, 2024 · e2dd6a8 · e2dd6a8
1 parent 22e9c5f
commit e2dd6a8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/croaring/src/treemap/iter.rs b/croaring/src/treemap/iter.rs
@@ -17,10 +17,11 @@ impl<'a> Iterator for To64Iter<'a> {
     }
 }
 
-fn to64iter<'a>(t: (&'a u32, &'a Bitmap)) -> To64Iter<'a> {
+fn to64iter<'a>((key, bitmap): (&'a u32, &'a Bitmap)) -> To64Iter<'a> {
+    assert!(!bitmap.is_empty(), "empty bitmap at {key}");
     To64Iter {
-        key: *t.0,
-        iterator: t.1.iter(),
+        key: *key,
+        iterator: bitmap.iter(),
     }
 }
 

diff --git a/fuzz/fuzz_targets/arbitrary_ops64/mod.rs b/fuzz/fuzz_targets/arbitrary_ops64/mod.rs
@@ -417,7 +417,7 @@ pub fn assert_64_eq(lhs: &Bitmap64, rhs: &Treemap) {
             let r = rhs.next().unwrap();
             assert_eq!(l, r, "{l} != {r} at {i}");
         }
-        assert!(rhs.next().is_none());
+        assert_eq!(rhs.next(), None);
         panic!("Serialize not equal, but all items equal?")
     }
 }