Skip to content

Commit

Permalink
format
Browse files Browse the repository at this point in the history
  • Loading branch information
SpectraL519 committed Mar 24, 2024
1 parent bdeb347 commit 14ca2ec
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 15 deletions.
11 changes: 4 additions & 7 deletions include/ap/argument_parser.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -1188,7 +1188,8 @@ class argument_parser {
* @param args Vector of default positional argument categories.
* @return Reference to the argument parser.
*/
argument_parser& default_positional_arguments(const std::vector<default_argument::positional>& args) noexcept {
argument_parser& default_positional_arguments(const std::vector<default_argument::positional>& args
) noexcept {
for (const auto arg : args)
this->_add_default_positional_argument(arg);
return *this;
Expand Down Expand Up @@ -1435,8 +1436,6 @@ class argument_parser {
case default_argument::positional::output:
this->add_positional_argument("output").help("Output file path");
break;

// TODO: throw error when discriminator is invalid
}
}

Expand Down Expand Up @@ -1473,8 +1472,6 @@ class argument_parser {
case default_argument::optional::multi_output:
this->add_optional_argument("output", "o").required().nargs(ap::nargs::at_least(1)).help("Output files paths");
break;

// TODO: throw error when discriminator is invalid
}
}

Expand Down Expand Up @@ -1595,10 +1592,10 @@ class argument_parser {
std::string value = argv[i];
if (this->_is_flag(value)) {
this->_strip_flag_prefix(value);
args.push_back(cmd_argument{cmd_argument::type_discriminator::flag, value});
args.push_back(cmd_argument{ cmd_argument::type_discriminator::flag, value });
}
else {
args.push_back(cmd_argument{cmd_argument::type_discriminator::value, value});
args.push_back(cmd_argument{ cmd_argument::type_discriminator::value, value });
}
}

Expand Down
4 changes: 1 addition & 3 deletions test/include/optional_argument_test_fixture.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,7 @@ struct optional_argument_test_fixture {
}

template <valid_argument_value_type T>
optional_argument<T>& sut_set_choices(
optional_argument<T>& sut, const std::vector<value_type<T>>& choices
) const {
optional_argument<T>& sut_set_choices(optional_argument<T>& sut, const std::vector<value_type<T>>& choices) const {
return sut.choices(choices);
}

Expand Down
7 changes: 2 additions & 5 deletions test/include/positional_argument_test_fixture.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,7 @@ struct positional_argument_test_fixture {
}

template <valid_argument_value_type T>
[[nodiscard]] const std::optional<std::string>& sut_get_help(const positional_argument<T>& sut
) const {
[[nodiscard]] const std::optional<std::string>& sut_get_help(const positional_argument<T>& sut) const {
return sut.help();
}

Expand Down Expand Up @@ -61,9 +60,7 @@ struct positional_argument_test_fixture {
}

template <valid_argument_value_type T>
positional_argument<T>& sut_set_choices(
positional_argument<T>& sut, const std::vector<value_type<T>>& choices
) const {
positional_argument<T>& sut_set_choices(positional_argument<T>& sut, const std::vector<value_type<T>>& choices) const {
return sut.choices(choices);
}

Expand Down

0 comments on commit 14ca2ec

Please sign in to comment.