Fixed clippy warnings.

mlaveaux · Sep 25, 2023 · 89d471d · 89d471d
1 parent 281452c
commit 89d471d
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/benchmarks/benches/vec_benchmarks.rs b/benchmarks/benches/vec_benchmarks.rs
@@ -1,6 +1,6 @@
 use std::{
     hint::black_box,
-    sync::{atomic::AtomicBool, Arc, Barrier, RwLock, Mutex},
+    sync::{atomic::AtomicBool, Arc, Barrier},
     thread::{self},
 };
 
@@ -135,7 +135,7 @@ pub fn benchmark_vector(c: &mut Criterion) {
                     black_box(x.vector.push(1));
                 },
                 num_threads,
-                10000,
+                NUM_ITERATIONS,
                 read_ratio);
         }
     }

diff --git a/bf-vec/src/lib.rs b/bf-vec/src/lib.rs
@@ -72,6 +72,10 @@ impl<T> BfVec<T> {
         self.shared.read().unwrap().len.load(Ordering::Relaxed)
     }
 
+    pub fn is_empty(&self) -> bool {
+        self.len() == 0
+    }
+
     /// Allocate the vector to be twice as long.
     fn reserve(&self, capacity: usize) {
         let mut write = self.shared.write().unwrap();
@@ -98,6 +102,12 @@ impl<T> BfVec<T> {
     }
 }
 
+impl<T> Default for BfVec<T> {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 impl<T> Drop for BfVecShared<T> {
     fn drop(&mut self) {
 

diff --git a/xtask/src/sanitizer.rs b/xtask/src/sanitizer.rs
@@ -8,6 +8,7 @@ use std::error::Error;
 pub use duct::cmd;
 
 #[cfg(not(unix))]
+#[allow(clippy::ptr_arg)]
 fn add_target_flag(_arguments: &mut Vec<String>) {}
 
 #[cfg(target_os = "linux")]