From 7a10f71ee564a7c1054683929f6a0110b0fa3b56 Mon Sep 17 00:00:00 2001 From: James Sully Date: Fri, 9 May 2025 16:58:25 +1000 Subject: refactor(main): eliminate a `mut` from config load code in main I think this makes for marginally better readability, since you don't have to wonder whether config_errored is set anywhere else. It's also slightly terser. --- src/main.rs | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/main.rs b/src/main.rs index 018f8d3f..401db23a 100644 --- a/src/main.rs +++ b/src/main.rs @@ -161,12 +161,10 @@ fn main() -> Result<(), Box> { } } - let mut config_errored = false; - let mut config = Config::load(&path) - .map_err(|err| { - warn!("{err:?}"); - config_errored = true; - }) + let config_load_result = Config::load(&path); + let config_errored = config_load_result.is_err(); + let mut config = config_load_result + .map_err(|err| warn!("{err:?}")) .unwrap_or_default(); let spawn_at_startup = mem::take(&mut config.spawn_at_startup); -- cgit