Skip to content
This repository has been archived by the owner on Jan 1, 2022. It is now read-only.

Commit

Permalink
Removed some unnecessary builder methods
Browse files Browse the repository at this point in the history
  • Loading branch information
savish committed Jul 20, 2018
1 parent d7f5484 commit 48f83e0
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions src/derives/clap.rs
Original file line number Diff line number Diff line change
Expand Up @@ -100,15 +100,13 @@ fn gen_app_augmentation(

// @TODO remove unneccessary builders
let modifier = match ty {
Ty::Bool => quote!( .takes_value(false).multiple(false) ),
Ty::Option => quote!( .takes_value(true).multiple(false) #validator ),
Ty::Bool => quote!(),
Ty::Option => quote!( .takes_value(true) #validator ),
Ty::Vec => quote!( .takes_value(true).multiple(true) #validator ),
Ty::Other if occurences => {
quote!( .takes_value(false).multiple_occurrences(true) )
}
Ty::Other if occurences => quote!( .multiple_occurrences(true) ),
Ty::Other => {
let required = !attrs.has_method("default_value");
quote!( .takes_value(true).multiple(false).required(#required) #validator )
quote!( .takes_value(true).required(#required) #validator )
}
};
let methods = attrs.methods();
Expand Down

0 comments on commit 48f83e0

Please sign in to comment.