From f536d5e2da1b267b4732ef544f6296d07bf4801a Mon Sep 17 00:00:00 2001 From: grainme Date: Fri, 24 Jan 2025 23:53:45 +0100 Subject: [PATCH 1/2] fix: improve error handling in main function - Replace unwrap() with some error handling - Print user-friendly error message - Set exit code on failure This change makes the tool more user-friendly by providing error messages instead of panicking. --- src/main.rs | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/main.rs b/src/main.rs index 4fc32f29..88e8ac65 100644 --- a/src/main.rs +++ b/src/main.rs @@ -854,7 +854,11 @@ fn main() { .enable_all() .build() .unwrap(); - rt.block_on(run()).unwrap(); + + if let Err(e) = rt.block_on(run()) { + eprintln!("Error: {}", e); + std::process::exit(1); + } } enum WorkMode { From d3ddf5b9f4bf5fe130e3ddd531be769a11751ecb Mon Sep 17 00:00:00 2001 From: grainme Date: Sat, 25 Jan 2025 11:10:21 +0100 Subject: [PATCH 2/2] use libc::EXIT_FAILURE for consistency instead of 1 --- src/main.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main.rs b/src/main.rs index 88e8ac65..baf45404 100644 --- a/src/main.rs +++ b/src/main.rs @@ -857,7 +857,7 @@ fn main() { if let Err(e) = rt.block_on(run()) { eprintln!("Error: {}", e); - std::process::exit(1); + std::process::exit(libc::EXIT_FAILURE); } }