From fdaf4bfe1d8b59b397b069429b48c655bfc358a8 Mon Sep 17 00:00:00 2001 From: Dan Davison Date: Tue, 18 Jul 2023 21:58:42 -0700 Subject: Revert "Refactor subcommand handling (#1467)" Fixes #1475 This reverts commit 8d14a1eba3ca93d6f6ebf3be541b37c6677139a6. --- src/main.rs | 42 +++++++++++++++++++++--------------------- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/src/main.rs b/src/main.rs index ba471102..a1f12622 100644 --- a/src/main.rs +++ b/src/main.rs @@ -82,24 +82,31 @@ fn run_app() -> std::io::Result { assets, ); - if let Err(error) = if opt.list_languages { - list_languages() + let subcommand_result = if opt.list_languages { + Some(list_languages()) } else if opt.list_syntax_themes { - subcommands::list_syntax_themes::list_syntax_themes() + Some(subcommands::list_syntax_themes::list_syntax_themes()) } else if opt.show_syntax_themes { - subcommands::show_syntax_themes::show_syntax_themes() + Some(subcommands::show_syntax_themes::show_syntax_themes()) } else if opt.show_themes { - subcommands::show_themes::show_themes(opt.dark, opt.light, opt.computed.is_light_mode) + Some(subcommands::show_themes::show_themes( + opt.dark, + opt.light, + opt.computed.is_light_mode, + )) } else if opt.show_colors { - subcommands::show_colors::show_colors() + Some(subcommands::show_colors::show_colors()) } else if opt.parse_ansi { - subcommands::parse_ansi::parse_ansi() + Some(subcommands::parse_ansi::parse_ansi()) } else { - Ok(()) - } { - match error.kind() { - ErrorKind::BrokenPipe => {} - _ => fatal(format!("{error}")), + None + }; + if let Some(result) = subcommand_result { + if let Err(error) = result { + match error.kind() { + ErrorKind::BrokenPipe => {} + _ => fatal(format!("{error}")), + } } return Ok(0); }; @@ -107,17 +114,10 @@ fn run_app() -> std::io::Result { let _show_config = opt.show_config; let config = config::Config::from(opt); - if let Err(error) = if _show_config { + if _show_config { let stdout = io::stdout(); let mut stdout = stdout.lock(); - subcommands::show_config::show_config(&config, &mut stdout) - } else { - Ok(()) - } { - match error.kind() { - ErrorKind::BrokenPipe => {} - _ => fatal(format!("{error}")), - } + subcommands::show_config::show_config(&config, &mut stdout)?; return Ok(0); } -- cgit v1.2.3