Skip to content

Commit

Permalink
Retrieve changes on 37a1f3b
Browse files Browse the repository at this point in the history
  • Loading branch information
hokacci committed Jun 21, 2022
1 parent b869b5a commit 5d6544a
Showing 1 changed file with 6 additions and 11 deletions.
17 changes: 6 additions & 11 deletions include/argparse/argparse.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -584,17 +584,12 @@ class Argument {
*/
void validate() const {
if (m_is_optional) {
if (m_is_used && !m_num_args_range.contains(m_values.size()) && !m_is_repeatable &&
!m_default_value.has_value()) {
throw_nargs_range_validation_error();
} else {
// TODO: check if an implicit value was programmed for this argument
if (!m_is_used && !m_default_value.has_value() && m_is_required) {
throw_required_arg_not_used_error();
}
if (m_is_used && m_is_required && m_values.size() == 0) {
throw_required_arg_no_value_provided_error();
}
// TODO: check if an implicit value was programmed for this argument
if (!m_is_used && !m_default_value.has_value() && m_is_required) {
throw_required_arg_not_used_error();
}
if (m_is_used && m_is_required && m_values.size() == 0) {
throw_required_arg_no_value_provided_error();
}
} else {
if (!m_num_args_range.contains(m_values.size()) && !m_default_value.has_value()) {
Expand Down

0 comments on commit 5d6544a

Please sign in to comment.