Long case insensitive style also makes short case insensitive.
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
Research direction
Start by reproducing the issue with the provided command_line_parser example, especially ./test -A, and trace how long_case_insensitive is handled in the command-line style parsing path. Done means long_case_insensitive no longer makes short options case-insensitive, with the 1.67.0 expected output preserved and a regression test covering the behavior.
Written by the indexing model from the issue text.
Description
When you use the ::long_case_insensitive style it inadvertently behaves as if you also selected ::short_case_insensitive.
I noticed this behavior since boost 1.68.0, compiling with 1.67.0 gives the correct result.
In v1.67.0, if you call the test program below like this, ./test -A, it will give the correct result:
Adam is selected: false
Alice is selected: true
In v1.68.0 and v1.69.0 it will give the wrong result, as if ::short_case_insensitive was also selected.:
Adam is selected: true
Alice is selected: false
Basically the -A argument becomes an -a argument.
Example code to illustrate the problem.
#include <iostream>
#include <boost/program_options.hpp>
namespace po = boost::program_options;
namespace po_style = boost::program_options::command_line_style;
int main(int argc, char* argv[]){
bool adamSelected = false;
bool aliceSelected = false;
try{
po::options_description desc("Allowed options");
desc.add_options()
("adam,a", po::value<bool>(&adamSelected)->default_value(false)->implicit_value(true),"Choose Adam")
("alice,A", po::value<bool>(&aliceSelected)->default_value(false)->implicit_value(true),"Choose Alice")
;
po::variables_map vm;
auto parsed = po::command_line_parser(argc, argv).options(desc).style(po_style::unix_style | po_style::long_case_insensitive).run();
po::store(parsed, vm);
po::notify(vm);
std::cout << "Adam is selected: " << (adamSelected?"true":"false") << std::endl
<< "Alice is selected: " << (aliceSelected?"true":"false") << std::endl;
}
catch(std::exception& e){
std::cerr << e.what() << std::endl;
}
return 0;
}
- Dominant language
- C++
- Stars
- 136
- Forks
- 117
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from boostorg/program_options
-
Provide C++20 module Open
Difficulty 5/5 Over a week Newbie friendliness 45/100
boostorg/program_options#160 · 2 comments ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
boostorg/program_options#159 · 3 comments ·
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
boostorg/program_options#147 ·
-
Difficulty 5/5 Over a week Newbie friendliness 30/100
boostorg/program_options#146 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
boostorg/program_options#144 ·
All issues in boostorg/program_options
Similar issues
-
ai_reviewed
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
ydb-platform/ydb#53869 · 3 comments ·
-
bug cert blocker needs triage
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
project-chip/connectedhomeip#74373 ·
-
upstream update
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
conan-io/conan-center-index#31035 ·
-
Bug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
documentation
Difficulty 1/5 Under an hour Newbie friendliness 85/100
vllm-project/vllm-ascend#17329 ·