OilfieldToolsNet/opm-common
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixes some problems with option parsing.

Browse Source 
Namely, that m requires an argument and not n as indicated to
getopt. This caused the help message to be printed whenever users
did not want exceptions to be thrown.

Additionally we fix the error message for options that need an argument
but do not get one.
This commit is contained in:
Markus Blatt 2017-04-19 15:14:25 +02:00
parent 621b94235c
commit d50ccf507d
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
examples/test_util/compareSummary.cpp
View File

@ -88,7 +88,7 @@ int main (int argc, char ** argv){
//------------------------------------------------
//For setting the options selected
while ((c = getopt(argc, argv, "dghik:Kmn:pP:rRs:vV:")) != -1) {
while ((c = getopt(argc, argv, "dghik:Km:npP:rRs:vV:")) != -1) {
switch (c) {
case 'd':
throwExceptionForTooGreatErrorRatio = false;
@ -146,8 +146,9 @@ int main (int argc, char ** argv){
keyword = optarg;
break;
case '?':
if (optopt == 'k' || optopt == 'm' || optopt == 's') {
std::cout << "Option requires an keyword." << std::endl;
if (optopt == 'k' || optopt == 'm' || optopt == 'P'
|| optopt == 's' || optopt == 'V') {
std::cout << "Option -"<<optopt<<" requires an keyword." << std::endl;
return EXIT_FAILURE;
}
else {