RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-29 20:17:45 +00:00
Code Issues Activity

core: remove unnecessary if statement in update control

Browse source
This commit is contained in:
John Shin 2023-05-02 12:49:49 -07:00
parent 460d346069
commit 3b8f3d04f4
1 changed files with 6 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/uucore/src/lib/mods/update_control.rs
View file

@ -122,7 +122,6 @@ pub mod arguments {
/// assert_eq!(update_mode, UpdateMode::ReplaceAll) /// assert_eq!(update_mode, UpdateMode::ReplaceAll)
/// } /// }
pub fn determine_update_mode(matches: &ArgMatches) -> UpdateMode { pub fn determine_update_mode(matches: &ArgMatches) -> UpdateMode {
if matches.contains_id(arguments::OPT_UPDATE) {
if let Some(mode) = matches.get_one::<String>(arguments::OPT_UPDATE) { if let Some(mode) = matches.get_one::<String>(arguments::OPT_UPDATE) {
match mode.as_str() { match mode.as_str() {
"all" => UpdateMode::ReplaceAll, "all" => UpdateMode::ReplaceAll,
@ -130,9 +129,6 @@ pub fn determine_update_mode(matches: &ArgMatches) -> UpdateMode {
"older" => UpdateMode::ReplaceIfOlder, "older" => UpdateMode::ReplaceIfOlder,
_ => unreachable!("other args restricted by clap"), _ => unreachable!("other args restricted by clap"),
} }
} else {
unreachable!("other args restricted by clap")
}
} else if matches.get_flag(arguments::OPT_UPDATE_NO_ARG) { } else if matches.get_flag(arguments::OPT_UPDATE_NO_ARG) {
// short form of this option is equivalent to using --update=older // short form of this option is equivalent to using --update=older
UpdateMode::ReplaceIfOlder UpdateMode::ReplaceIfOlder