Skip to content

Commit

Permalink
fix(derive): dedup subcommands
Browse files Browse the repository at this point in the history
  • Loading branch information
rkrasiuk committed Feb 27, 2023
1 parent 03ee714 commit 7229ca6
Showing 1 changed file with 7 additions and 10 deletions.
17 changes: 7 additions & 10 deletions clap_derive/src/derives/subcommand.rs
Expand Up @@ -255,13 +255,11 @@ fn gen_augment(
let subcommand = quote! {
let #app_var = #app_var.subcommand({
#deprecations;
let #subcommand_var = clap::Command::new(#name);
let #subcommand_var = #subcommand_var
let #subcommand_var = clap::Command::new(#name)
.subcommand_required(true)
.arg_required_else_help(true);
let #subcommand_var = #subcommand_var #initial_app_methods;
let #subcommand_var = #arg_block;
#subcommand_var #final_from_attrs #override_methods
.arg_required_else_help(true)
#initial_app_methods;
#arg_block #final_from_attrs #override_methods
});
};
Some(subcommand)
Expand All @@ -283,13 +281,12 @@ fn gen_augment(
args::gen_augment(&fields, &subcommand_var, item, override_required)
}
Unit => {
let arg_block = quote!( #subcommand_var );
let initial_app_methods = item.initial_top_level_methods();
let final_from_attrs = item.final_top_level_methods();
quote! {
let #subcommand_var = #subcommand_var #initial_app_methods;
let #subcommand_var = #arg_block;
#subcommand_var #final_from_attrs
#subcommand_var
#initial_app_methods
#final_from_attrs
}
},
Unnamed(FieldsUnnamed { ref unnamed, .. }) if unnamed.len() == 1 => {
Expand Down

0 comments on commit 7229ca6

Please sign in to comment.