diff options
author | Jiayi Zhao <jeff.no.zhao@gmail.com> | 2020-12-13 12:42:53 -0500 |
---|---|---|
committer | Jiayi Zhao <jeff.no.zhao@gmail.com> | 2020-12-13 12:42:53 -0500 |
commit | 2e5cdd07c2d8c18bad4099c82e36af64dc473a64 (patch) | |
tree | a9d7dee0f5a603cc96d6415a4549436a0f1818e0 /src/io/io_observer.rs | |
parent | 468e607c5d1cd5d554ae356fd3c8e78425840347 (diff) |
code cleanup and fix path not showing when too long
Diffstat (limited to 'src/io/io_observer.rs')
-rw-r--r-- | src/io/io_observer.rs | 38 |
1 files changed, 14 insertions, 24 deletions
diff --git a/src/io/io_observer.rs b/src/io/io_observer.rs index 0648fbd..532b6f5 100644 --- a/src/io/io_observer.rs +++ b/src/io/io_observer.rs @@ -25,10 +25,7 @@ impl IOWorkerObserver { } pub fn join(self) -> bool { - match self.handle.join() { - Ok(_) => true, - _ => false, - } + matches!(self.handle.join(), Ok(_)) } pub fn set_progress(&mut self, progress: IOWorkerProgress) { self.progress = Some(progress); @@ -38,26 +35,19 @@ impl IOWorkerObserver { None => {} Some(progress) => { let size_str = format::file_size_to_string(progress.processed()); - match progress.kind() { - FileOp::Cut => { - let msg = format!( - "moving ({}/{}) {} completed", - progress.index() + 1, - progress.len(), - size_str - ); - self.msg = msg; - } - FileOp::Copy => { - let msg = format!( - "copying ({}/{}) {} completed", - progress.index() + 1, - progress.len(), - size_str - ); - self.msg = msg; - } - } + let op_str = match progress.kind() { + FileOp::Cut => "Moving", + FileOp::Copy => "Copying", + }; + + let msg = format!( + "{} ({}/{}) {} completed", + op_str, + progress.index() + 1, + progress.len(), + size_str + ); + self.msg = msg; } } } |