From d530d6e39d58f5759d2db0a9bda1ea5e21154f83 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bla=C5=BE=20Hrastnik?= Date: Sun, 11 Jul 2021 16:35:57 +0900 Subject: [PATCH] Further simplify error handling in :commands --- helix-term/src/commands.rs | 108 +++++++++++++------------------------ helix-view/src/editor.rs | 13 ++--- 2 files changed, 41 insertions(+), 80 deletions(-) diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs index 8a9ffb91f..8bbdfdfa5 100644 --- a/helix-term/src/commands.rs +++ b/helix-term/src/commands.rs @@ -23,7 +23,7 @@ use helix_view::{ Document, DocumentId, Editor, ViewId, }; -use anyhow::{anyhow, bail}; +use anyhow::{anyhow, bail, Context as _}; use helix_lsp::{ lsp, util::{lsp_pos_to_pos, lsp_range_to_range, pos_to_lsp_pos, range_to_lsp_range}, @@ -1197,16 +1197,9 @@ mod cmd { args: &[&str], _event: PromptEvent, ) -> anyhow::Result<()> { - match args.get(0) { - Some(path) => { - let _ = cx.editor.open(path.into(), Action::Replace)?; - - Ok(()) - } - None => { - bail!("wrong argument count"); - } - } + let path = args.get(0).context("wrong argument count")?; + let _ = cx.editor.open(path.into(), Action::Replace)?; + Ok(()) } fn write_impl>( @@ -1217,9 +1210,7 @@ mod cmd { let (_, doc) = current!(cx.editor); if let Some(path) = path { - if let Err(err) = doc.set_path(path.as_ref()) { - bail!("invalid filepath: {}", err); - }; + doc.set_path(path.as_ref()).context("invalid filepath")?; } if doc.path().is_none() { bail!("cannot write a buffer without a filename"); @@ -1306,14 +1297,11 @@ mod cmd { _ => None, }; - if let Some(s) = style { - let doc = doc_mut!(cx.editor); - doc.indent_style = s; + let style = style.context("invalid indent style")?; + let doc = doc_mut!(cx.editor); + doc.indent_style = style; - Ok(()) - } else { - bail!("invalid indent style '{}'", args[0]); - } + Ok(()) } /// Sets or reports the current document's line ending setting. @@ -1352,12 +1340,9 @@ mod cmd { _ => None, }; - if let Some(le) = line_ending { - doc_mut!(cx.editor).line_ending = le; - Ok(()) - } else { - bail!("invalid line ending '{}'", args[0]); - } + let line_ending = line_ending.context("invalid line ending")?; + doc_mut!(cx.editor).line_ending = line_ending; + Ok(()) } fn earlier( @@ -1397,7 +1382,7 @@ mod cmd { event: PromptEvent, ) -> anyhow::Result<()> { let handle = write_impl(cx, args.first())?; - helix_lsp::block_on(handle)?; + let _ = helix_lsp::block_on(handle)?; quit(cx, &[], event) } @@ -1407,7 +1392,7 @@ mod cmd { event: PromptEvent, ) -> anyhow::Result<()> { let handle = write_impl(cx, args.first())?; - helix_lsp::block_on(handle)?; + let _ = helix_lsp::block_on(handle)?; force_quit(cx, &[], event) } @@ -1532,12 +1517,7 @@ mod cmd { args: &[&str], _event: PromptEvent, ) -> anyhow::Result<()> { - let theme = if let Some(theme) = args.first() { - theme - } else { - bail!("theme name not provided"); - }; - + let theme = args.first().context("theme not provided")?; cx.editor.set_theme_from_name(theme) } @@ -1598,10 +1578,7 @@ mod cmd { doc.append_changes_to_history(view.id); Ok(()) } - Err(e) => { - log::error!("Couldn't get system clipboard contents: {:?}", e); - bail!("Couldn't get system clipboard contents: {:?}", e); - } + Err(e) => Err(e.context("Couldn't get system clipboard contents")), } } @@ -1620,24 +1597,18 @@ mod cmd { args: &[&str], _event: PromptEvent, ) -> anyhow::Result<()> { - let dir = args - .first() - .ok_or_else(|| anyhow!("target directory not provided"))?; + let dir = args.first().context("target directory not provided")?; if let Err(e) = std::env::set_current_dir(dir) { bail!("Couldn't change the current working directory: {:?}", e); } - match std::env::current_dir() { - Ok(cwd) => { - cx.editor.set_status(format!( - "Current working directory is now {}", - cwd.display() - )); - Ok(()) - } - Err(e) => bail!("Couldn't get the new working directory: {}", e), - } + let cwd = std::env::current_dir().context("Couldn't get the new working directory")?; + cx.editor.set_status(format!( + "Current working directory is now {}", + cwd.display() + )); + Ok(()) } fn show_current_directory( @@ -1645,14 +1616,10 @@ mod cmd { _args: &[&str], _event: PromptEvent, ) -> anyhow::Result<()> { - match std::env::current_dir() { - Ok(cwd) => { - cx.editor - .set_status(format!("Current working directory is {}", cwd.display())); - Ok(()) - } - Err(e) => bail!("Couldn't get the current working directory: {}", e), - } + let cwd = std::env::current_dir().context("Couldn't get the new working directory")?; + cx.editor + .set_status(format!("Current working directory is {}", cwd.display())); + Ok(()) } /// Sets the [`Document`]'s encoding.. @@ -2870,10 +2837,10 @@ fn yank_joined_to_clipboard_impl(editor: &mut Editor, separator: &str) -> anyhow let joined = values.join(separator); - if let Err(e) = editor.clipboard_provider.set_contents(joined) { - log::error!("Couldn't set system clipboard content: {:?}", e); - bail!("Couldn't set system clipboard content: {:?}", e); - } + editor + .clipboard_provider + .set_contents(joined) + .context("Couldn't set system clipboard content")?; editor.set_status(msg); @@ -2894,7 +2861,6 @@ fn yank_main_selection_to_clipboard_impl(editor: &mut Editor) -> anyhow::Result< .fragment(doc.text().slice(..)); if let Err(e) = editor.clipboard_provider.set_contents(value.into_owned()) { - log::error!("Couldn't set system clipboard content: {:?}", e); bail!("Couldn't set system clipboard content: {:?}", e); } @@ -2965,10 +2931,7 @@ fn paste_clipboard_impl(editor: &mut Editor, action: Paste) -> anyhow::Result<() Ok(()) } Ok(None) => Ok(()), - Err(e) => { - log::error!("Couldn't get system clipboard contents: {:?}", e); - bail!("Couldn't get system clipboard contents: {:?}", e); - } + Err(e) => Err(e.context("Couldn't get system clipboard contents")), } } @@ -3000,7 +2963,7 @@ fn replace_with_yanked(cx: &mut Context) { } } -fn replace_selections_with_clipboard_impl(editor: &mut Editor) { +fn replace_selections_with_clipboard_impl(editor: &mut Editor) -> anyhow::Result<()> { let (view, doc) = current!(editor); match editor.clipboard_provider.get_contents() { @@ -3014,13 +2977,14 @@ fn replace_selections_with_clipboard_impl(editor: &mut Editor) { doc.apply(&transaction, view.id); doc.append_changes_to_history(view.id); + Ok(()) } - Err(e) => log::error!("Couldn't get system clipboard contents: {:?}", e), + Err(e) => Err(e.context("Couldn't get system clipboard contents")), } } fn replace_selections_with_clipboard(cx: &mut Context) { - replace_selections_with_clipboard_impl(&mut cx.editor); + let _ = replace_selections_with_clipboard_impl(&mut cx.editor); } fn paste_after(cx: &mut Context) { diff --git a/helix-view/src/editor.rs b/helix-view/src/editor.rs index c80535ed7..a468b87ca 100644 --- a/helix-view/src/editor.rs +++ b/helix-view/src/editor.rs @@ -98,14 +98,11 @@ impl Editor { } pub fn set_theme_from_name(&mut self, theme: &str) -> anyhow::Result<()> { - let theme = match self.theme_loader.load(theme.as_ref()) { - Ok(theme) => theme, - Err(e) => { - log::warn!("failed setting theme `{}` - {}", theme, e); - anyhow::bail!("failed setting theme `{}` - {}", theme, e); - } - }; - + use anyhow::Context; + let theme = self + .theme_loader + .load(theme.as_ref()) + .with_context(|| format!("failed setting theme `{}`", theme))?; self.set_theme(theme); Ok(()) }