Merge pull request #26 from manelmontilla/ineffectual-drops

wruster/thread_pool: remove ineffectual drops
manelmontilla · Nov 23, 2023 · 0957c8c · 0957c8c
2 parents a332b83 + f7b5d06
commit 0957c8c
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 19 deletions.
diff --git a/wruster/src/thread_pool/dynamic_pool.rs b/wruster/src/thread_pool/dynamic_pool.rs
@@ -152,15 +152,6 @@ impl Dynamic {
     }
 }
 
-impl Drop for Dynamic {
-    fn drop(&mut self) {
-        for worker in &*self.workers {
-            #[allow(clippy::drop_ref)]
-            drop(worker);
-        }
-    }
-}
-
 #[cfg(test)]
 mod tests {
     use super::*;

diff --git a/wruster/src/thread_pool/static_pool.rs b/wruster/src/thread_pool/static_pool.rs
@@ -109,16 +109,6 @@ impl Static {
     }
 }
 
-impl Drop for Static {
-    fn drop(&mut self) {
-        let workers = &self.workers;
-        for worker in workers {
-            #[allow(clippy::drop_ref)]
-            drop(worker);
-        }
-    }
-}
-
 #[cfg(test)]
 mod tests {
     use super::*;