fix: detect if space should be added before spec_vals

This commit is contained in:
knidarkness 2020-08-25 10:23:47 +03:00
parent 29df5c2d16
commit 530b179776
3 changed files with 12 additions and 11 deletions

View file

@ -504,7 +504,7 @@ impl<'help, 'app, 'parser, 'writer> Help<'help, 'app, 'parser, 'writer> {
} else { } else {
String::new() String::new()
}; };
let env_info = format!(" [env: {}{}]", env.0.to_string_lossy(), env_val); let env_info = format!("[env: {}{}]", env.0.to_string_lossy(), env_val);
spec_vals.push(env_info); spec_vals.push(env_info);
} }
if !a.is_set(ArgSettings::HideDefaultValue) && !a.default_vals.is_empty() { if !a.is_set(ArgSettings::HideDefaultValue) && !a.default_vals.is_empty() {
@ -520,7 +520,7 @@ impl<'help, 'app, 'parser, 'writer> Help<'help, 'app, 'parser, 'writer> {
.collect::<Vec<_>>() .collect::<Vec<_>>()
.join(" "); .join(" ");
spec_vals.push(format!(" [default: {}]", pvs)); spec_vals.push(format!("[default: {}]", pvs));
} }
if !a.aliases.is_empty() { if !a.aliases.is_empty() {
debug!("Help::spec_vals: Found aliases...{:?}", a.aliases); debug!("Help::spec_vals: Found aliases...{:?}", a.aliases);
@ -534,7 +534,7 @@ impl<'help, 'app, 'parser, 'writer> Help<'help, 'app, 'parser, 'writer> {
.join(", "); .join(", ");
if !als.is_empty() { if !als.is_empty() {
spec_vals.push(format!(" [aliases: {}]", als)); spec_vals.push(format!("[aliases: {}]", als));
} }
} }
@ -566,12 +566,13 @@ impl<'help, 'app, 'parser, 'writer> Help<'help, 'app, 'parser, 'writer> {
a.possible_vals a.possible_vals
); );
spec_vals.push(format!( spec_vals.push(format!("[possible values: {}]", a.possible_vals.join(", ")));
" [possible values: {}]",
a.possible_vals.join(", ")
));
} }
spec_vals.join(" ") let prefix = match !spec_vals.is_empty() && !a.get_about().unwrap_or("").is_empty() {
true => " ",
false => "",
};
prefix.to_string() + &spec_vals.join(" ")
} }
} }