From 237223ed58691a428a1c0330322fad7b25d3fafd Mon Sep 17 00:00:00 2001 From: joao-vitor-sr Date: Fri, 29 Sep 2023 03:50:18 +0000 Subject: [PATCH] Removes the panic, throws a warning & returns an empty list, which should be expected behavior (#73) * refactor(errors): better message when languaage directory doesn't exist * refactor(languages_errors): better approach to the languages directory --- src/main.rs | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/main.rs b/src/main.rs index 61c8f61..738eeab 100644 --- a/src/main.rs +++ b/src/main.rs @@ -191,8 +191,13 @@ fn main() -> crossterm::Result<()> { } if opt.list_languages { - opt.languages() - .expect("Couldn't get installed languages under config directory. Make sure the config directory exists.") + match opt.languages() { + Ok(languages) => languages, + Err(_) => { + println!("Warning: Couldn't get installed languages under config directory. Make sure the config directory exists."); + vec![] + } + } .iter() .for_each(|name| println!("{}", name.to_str().expect("Ill-formatted language name."))); return Ok(());