KeqingMoe/argparse
1
0
Fork 0
mirror of https://github.com/KeqingMoe/argparse.git synced 2025-07-04 15:14:39 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Move ArgumentParser::parse_args_validate logic into ::parse_args

Browse Source 
This code was previously moved from ::parse_args to its own method, but
has since been simplified.  Moving the actual work to Argument::validate
in commit 603e87ae6 leaves a single loop that fits back into ::parse_args.

Signed-off-by: Sean Robinson <sean.robinson@scottsdalecc.edu>
This commit is contained in:
Sean Robinson 2022-02-28 13:20:08 -07:00
parent 063d708c3e
commit 843e4eefb0
1 changed files with 4 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
include/argparse/argparse.hpp
View File

@ -958,7 +958,10 @@ public:
*/ */
void parse_args(const std::vector<std::string> &arguments) { void parse_args(const std::vector<std::string> &arguments) {
parse_args_internal(arguments); parse_args_internal(arguments);
parse_args_validate(); // Check if all arguments are parsed
for (const auto& [unused, argument] : m_argument_map) {
argument->validate();
}
} }
/* Main entry point for parsing command-line arguments using this /* Main entry point for parsing command-line arguments using this
@ -1134,16 +1137,6 @@ private:
m_is_parsed = true; m_is_parsed = true;
} }
/*
* @throws std::runtime_error in case of any invalid argument
*/
void parse_args_validate() {
// Check if all arguments are parsed
for (const auto& [unused, argument] : m_argument_map) {
argument->validate();
}
}
// Used by print_help. // Used by print_help.
std::size_t get_length_of_longest_argument() const { std::size_t get_length_of_longest_argument() const {
if (m_argument_map.empty()) { if (m_argument_map.empty()) {