Skip to content

Commit

Permalink
fix: fixes a bug where default values should have triggered a conditi…
Browse files Browse the repository at this point in the history
…onal requirement but didnt

Closes #831
  • Loading branch information
kbknapp committed Feb 3, 2017
1 parent 3e99887 commit a7a44ba
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/app/parser.rs
Original file line number Diff line number Diff line change
Expand Up @@ -995,6 +995,7 @@ impl<'a, 'b> Parser<'a, 'b>
-> ClapResult<()> {
debugln!("Parser::validate;");
let mut reqs_validated = false;
try!(self.add_defaults(matcher));
if let Some(a) = needs_val_of {
debugln!("Parser::validate: needs_val_of={:?}", a);
if let Some(o) = find_by_name!(self, &a, opts, iter) {
Expand All @@ -1018,7 +1019,6 @@ impl<'a, 'b> Parser<'a, 'b>
!reqs_validated {
try!(self.validate_required(matcher));
}
try!(self.add_defaults(matcher));
try!(self.validate_matched_args(matcher));
matcher.usage(self.create_usage(&[]));

Expand Down

0 comments on commit a7a44ba

Please sign in to comment.