style(clippy): Unnecessary move

Function is used in a single place and the move doesn't seem to provide
any safety benefits, so 💨
This commit is contained in:
Jade Ellis
2025-11-11 13:59:12 +00:00
committed by Ellis Git
parent b04f1332db
commit eaf6a889c2
+4 -3
View File
@@ -184,7 +184,7 @@ fn spawn_one(
let handle = thread::Builder::new() let handle = thread::Builder::new()
.name(WORKER_NAME.into()) .name(WORKER_NAME.into())
.stack_size(WORKER_STACK_SIZE) .stack_size(WORKER_STACK_SIZE)
.spawn(move || self.worker(id, recv))?; .spawn(move || self.worker(id, &recv))?;
workers.push(handle); workers.push(handle);
@@ -260,9 +260,9 @@ async fn execute(&self, queue: &Sender<Cmd>, cmd: Cmd) -> Result {
tid = ?thread::current().id(), tid = ?thread::current().id(),
), ),
)] )]
fn worker(self: Arc<Self>, id: usize, recv: Receiver<Cmd>) { fn worker(self: Arc<Self>, id: usize, recv: &Receiver<Cmd>) {
self.worker_init(id); self.worker_init(id);
self.worker_loop(&recv); self.worker_loop(recv);
} }
#[implement(Pool)] #[implement(Pool)]
@@ -375,6 +375,7 @@ fn handle_iter(&self, mut cmd: Seek) {
keys = %cmd.key.len(), keys = %cmd.key.len(),
), ),
)] )]
fn handle_batch(self: &Arc<Self>, mut cmd: Get) { fn handle_batch(self: &Arc<Self>, mut cmd: Get) {
debug_assert!(cmd.key.len() > 1, "should have more than one key"); debug_assert!(cmd.key.len() > 1, "should have more than one key");
debug_assert!(!cmd.key.iter().any(SmallVec::is_empty), "querying for empty key"); debug_assert!(!cmd.key.iter().any(SmallVec::is_empty), "querying for empty key");