tyler
/
proxmox-backup
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

getopt: cleanup: don't condense 'if' statements too much 

In a language which enforces curly braces this looks weird
and rustfmt doesn't like it.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
This commit is contained in:
Wolfgang Bumiller 2018-12-28 14:24:24 +01:00
parent 7d95c10da0
commit 2767c5d39b
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/getopts.rs
View File

@ -79,7 +79,9 @@ pub fn parse_arguments<T: AsRef<str>>(
if let Schema::Boolean(boolean_schema) = param_schema.as_ref() { if let Schema::Boolean(boolean_schema) = param_schema.as_ref() {
want_bool = true; want_bool = true;
if let Some(default) = boolean_schema.default { if let Some(default) = boolean_schema.default {
if default == false { can_default = true; } if default == false {
can_default = true;
}
} else { } else {
can_default = true; can_default = true;
} }
@ -93,7 +95,9 @@ pub fn parse_arguments<T: AsRef<str>>(
let next = args[pos + 1].as_ref(); let next = args[pos + 1].as_ref();
if let RawArgument::Argument { value: _ } = parse_argument(next) { if let RawArgument::Argument { value: _ } = parse_argument(next) {
next_is_argument = true; next_is_argument = true;
if let Ok(_) = parse_boolean(next) { next_is_bool = true; } if let Ok(_) = parse_boolean(next) {
next_is_bool = true;
}
} }
} }
@ -144,7 +148,9 @@ pub fn parse_arguments<T: AsRef<str>>(
} }
} }
if errors.len() > 0 { return Err(errors); } if errors.len() > 0 {
return Err(errors);
}
if arg_param.len() > 0 { if arg_param.len() > 0 {
rest = rest[arg_param.len()..].to_vec(); rest = rest[arg_param.len()..].to_vec();