yate: clippy ftw

Signed-off-by: aserowy <serowy@hotmail.com>
aserowy · Feb 18, 2024 · 407f6f2 · 407f6f2
1 parent 272f070
commit 407f6f2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/y1337-frontend/src/tui.rs b/y1337-frontend/src/tui.rs
@@ -84,11 +84,8 @@ pub async fn run(settings: Settings) -> Result<(), AppError> {
                 PostRenderAction::Open(path) => {
                     let path = path.clone();
                     // FIX: open files using cli programs sets those in background and y gets stuck
-                    match open::that(path) {
-                        Ok(_) => (),
-                        Err(_err) => {
-                            // TODO: log error
-                        }
+                    if let Err(_err) = open::that(path) {
+                        // TODO: log error
                     }
                 }
                 PostRenderAction::Quit(stdout_result) => {

diff --git a/y1337/src/main.rs b/y1337/src/main.rs
@@ -45,5 +45,5 @@ fn cli() -> Command {
 
 fn map_args_to_settings(args: &ArgMatches, settings: &mut Settings) {
     settings.stdout_on_open = args.get_flag("stdout-on-open");
-    settings.startup_path = args.get_one("path").map(|path: &PathBuf| path.clone());
+    settings.startup_path = args.get_one("path").cloned();
 }