Skip to content

Commit

Permalink
nit: rename fn 'join_all' & make it private
Browse files Browse the repository at this point in the history
  • Loading branch information
claddyy committed Dec 12, 2024
1 parent e75d200 commit 1ca49c6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/maker/api.rs
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ impl Default for ThreadPool {

impl Drop for ThreadPool {
fn drop(&mut self) {
if let Err(e) = self.join_all() {
if let Err(e) = self.join_all_threads() {
log::error!("Error joining threads in via drop: {:?}", e);
}
}
Expand All @@ -121,7 +121,7 @@ impl ThreadPool {
threads.push(handle);
}

pub fn join_all(&self) -> Result<(), MakerError> {
fn join_all_threads(&self) -> Result<(), MakerError> {
let mut threads = self
.threads
.lock()
Expand Down

0 comments on commit 1ca49c6

Please sign in to comment.