From a3e87f5a0333d7ff0803fad8cf1b5cd93818427d Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Thu, 17 Oct 2024 16:51:39 +0200 Subject: [PATCH] client: progress log: small opinionated code clean-up It was fine as is, but IMO saving a few lines is nice, albeit it makes the atomic fetch expressions look slightly complexer by wrapping them directly with the HumanByte and TimeSpan from-constructors. Signed-off-by: Thomas Lamprecht --- pbs-client/src/backup_writer.rs | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/pbs-client/src/backup_writer.rs b/pbs-client/src/backup_writer.rs index 4d2e3d08..4d2e8a80 100644 --- a/pbs-client/src/backup_writer.rs +++ b/pbs-client/src/backup_writer.rs @@ -691,16 +691,11 @@ impl BackupWriter { loop { tokio::time::sleep(tokio::time::Duration::from_secs(60)).await; - let size = stream_len3.load(Ordering::SeqCst); - let size_uploaded = uploaded_len.load(Ordering::SeqCst); - let elapsed = start_time.elapsed(); + let size = HumanByte::from(stream_len3.load(Ordering::SeqCst)); + let size_uploaded = HumanByte::from(uploaded_len.load(Ordering::SeqCst)); + let elapsed = TimeSpan::from(start_time.elapsed()); - log::info!( - " processed {} in {}, uploaded {}", - HumanByte::from(size), - TimeSpan::from(elapsed), - HumanByte::from(size_uploaded), - ); + log::info!("processed {size} in {elapsed}, uploaded {size_uploaded}"); } })) } else {