Skip to content

Restore start_index option #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
pavelkryukov wants to merge 14 commits into
base: develop
Choose a base branch
from
Open

Restore start_index option #19

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
7811b18
add xcode and clang
badaix Jun 26, 2018
157957a
Add start_index argument to 'parse' function
pavelkryukov Sep 11, 2018
4b792ac
Merge pull request #15 from pavelkryukov/develop
badaix Oct 4, 2018
011f92f
Update README.md
badaix Oct 4, 2018
6ddaf3f
merge with master
badaix Oct 6, 2019
69640d8
add parsing of ini files
badaix Oct 6, 2019
6c3a47a
add test infrastructure
badaix Nov 17, 2019
d3fb96a
fix test binary name
badaix Nov 17, 2019
8ecefcb
add test for config files
badaix Nov 17, 2019
2ffa856
add overload for bool value
badaix Nov 17, 2019
60cc633
raise warning level
badaix Nov 17, 2019
88d02d3
Restore start_index option
pavelkryukov Apr 15, 2020
a604af2
Merge branch 'develop' into develop2
pavelkryukov Feb 13, 2022
f507d0a
Update popl.hpp
pavelkryukov Feb 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions include/popl.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -339,7 +339,8 @@ class OptionParser
/// Parse the command line into the added Options
/// @param argc command line argument count
/// @param argv command line arguments
void parse(int argc, const char* const argv[]);
/// @param start_index index of starting argument
void parse(int argc, const char * const argv[], int start_index = 1);

/// Delete all parsed options
void reset();
Expand Down Expand Up @@ -993,9 +994,9 @@ inline void OptionParser::parse(const std::string& ini_filename)
}
}

inline void OptionParser::parse(int argc, const char* const argv[])
inline void OptionParser::parse(int argc, const char* const argv[], int start_index)
{
for (int n = 1; n < argc; ++n)
for (int n = start_index; n < argc; ++n)
{
const std::string arg(argv[n]);
if (arg == "--")
Expand Down
14 changes: 14 additions & 0 deletions test/test_main.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,3 +54,17 @@ TEST_CASE("config file")
}
}

TEST_CASE("start index")
{
OptionParser op("Allowed options");
auto help_option = op.add<Switch>("h", "help", "produce help message");
auto int_option = op.add<Value<int>>("i", "int", "test for int value", 42);
std::vector<const char*> args = {"popl", "-h", "--", "-i", "43"};

op.parse(args.size(), args.data(), 3);
REQUIRE(help_option->count() == 0);
REQUIRE(int_option->is_set());
REQUIRE(int_option->count() == 1);
REQUIRE(int_option->value() == 43);
}