node: simplify upload-pack reader joining

The `join` method is equivalent to the custom loop that was written for joining
the `reader` thread in upload-pack.

Signed-off-by: Fintan Halpenny <fintan.halpenny@gmail.com>
X-Clacks-Overhead: GNU Terry Pratchett
This commit is contained in:
Fintan Halpenny 2024-05-01 08:53:53 +01:00 committed by cloudhead
parent 740106d759
commit de434bb437
No known key found for this signature in database
1 changed files with 3 additions and 7 deletions

View File

@ -124,14 +124,10 @@ where
// N.b. we only care if the `reader` is finished. We then kill
// the child which will end the thread for the sender.
loop {
if reader.is_finished() {
child.kill()?;
break;
} else {
std::thread::sleep(std::time::Duration::from_millis(100));
}
if let Err(e) = reader.join() {
log::warn!(target: "worker", "Upload pack thread panicked: {e:?}");
}
child.kill()?;
Ok::<_, io::Error>(())
})?;