diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/commands/open_file.rs | 23 | ||||
-rw-r--r-- | src/commands/zoxide.rs | 24 | ||||
-rw-r--r-- | src/key_command/impl_appexecute.rs | 2 |
3 files changed, 21 insertions, 28 deletions
diff --git a/src/commands/open_file.rs b/src/commands/open_file.rs index 48c0d0b..24acae1 100644 --- a/src/commands/open_file.rs +++ b/src/commands/open_file.rs @@ -72,11 +72,11 @@ where S: AsRef<std::ffi::OsStr>, { if option.get_fork() { - let (child_id, handle) = fork_execute(&option, files, context.clone_event_tx())?; + let (child_id, handle) = fork_execute(option, files, context.clone_event_tx())?; context.worker_context_mut().push_child(child_id, handle); } else { backend.terminal_drop(); - execute_and_wait(&option, files)?; + execute_and_wait(option, files)?; backend.terminal_restore()?; } Ok(()) @@ -139,20 +139,17 @@ where } Ok(n) => { let option = &options[n]; - open_with_entry(context, backend, option, &files)?; + open_with_entry(context, backend, option, files)?; } Err(_) => { let mut args_iter = user_input.split_whitespace(); - match args_iter.next() { - Some(cmd) => { - backend.terminal_drop(); - let mut option = AppMimetypeEntry::new(String::from(cmd)); - option.args(args_iter); - let res = execute_and_wait(&option, files); - backend.terminal_restore()?; - res? - } - None => {} + if let Some(cmd) = args_iter.next() { + backend.terminal_drop(); + let mut option = AppMimetypeEntry::new(String::from(cmd)); + option.args(args_iter); + let res = execute_and_wait(&option, files); + backend.terminal_restore()?; + res? } } } diff --git a/src/commands/zoxide.rs b/src/commands/zoxide.rs index 1abe9f2..9ee1bf6 100644 --- a/src/commands/zoxide.rs +++ b/src/commands/zoxide.rs @@ -27,14 +27,12 @@ pub fn zoxide_query(context: &mut AppContext, args: &str) -> JoshutoResult { context .message_queue_mut() .push_info(format!("z {:?}", zoxide_path)); - change_directory::change_directory(context, &path)?; - } - } else { - if let Ok(zoxide_str) = std::str::from_utf8(&zoxide_output.stderr) { - context - .message_queue_mut() - .push_error(zoxide_str.to_string()); + change_directory::change_directory(context, path)?; } + } else if let Ok(zoxide_str) = std::str::from_utf8(&zoxide_output.stderr) { + context + .message_queue_mut() + .push_error(zoxide_str.to_string()); } Ok(()) } @@ -65,14 +63,12 @@ pub fn zoxide_query_interactive( context .message_queue_mut() .push_info(format!("zi {:?}", zoxide_path)); - change_directory::change_directory(context, &path)?; - } - } else { - if let Ok(zoxide_str) = std::str::from_utf8(&zoxide_output.stderr) { - context - .message_queue_mut() - .push_error(zoxide_str.to_string()); + change_directory::change_directory(context, path)?; } + } else if let Ok(zoxide_str) = std::str::from_utf8(&zoxide_output.stderr) { + context + .message_queue_mut() + .push_error(zoxide_str.to_string()); } Ok(()) } diff --git a/src/key_command/impl_appexecute.rs b/src/key_command/impl_appexecute.rs index 993d3f8..bf2b9fc 100644 --- a/src/key_command/impl_appexecute.rs +++ b/src/key_command/impl_appexecute.rs @@ -106,7 +106,7 @@ impl AppExecute for Command { Self::SearchFzf => search_fzf::search_fzf(context, backend), Self::SubdirFzf => subdir_fzf::subdir_fzf(context, backend), - Self::Zoxide(arg) => zoxide::zoxide_query(context, &arg), + Self::Zoxide(arg) => zoxide::zoxide_query(context, arg), Self::ZoxideInteractive => zoxide::zoxide_query_interactive(context, backend), } } |