Method of using defaut argument still fails about argagg HOT 4 CLOSED

This is actually intended behavior and has been brought up before in #16. Here's an edit of my response:

Yea, my concern with having that style of optional argument is that it actually introduces a third state to the flag.

• flag not specified
• flag specified followed by non-flag
• flag specified followed by a flag (and thus specified without argument)

Currently a flag has two states:

• flag not specified
• flag specified with argument

I feel like if there's some default behavior you want with a flag specified with no argument it can be captured by its own flag. Consider git log for example, --oneline is equivalent to --pretty=oneline.

Also, it messes with the current greedy argument behavior. For example, foo --log --verbose would make --verbose an argument to --log.

I'm open to further discussion on this though.

from argagg.

Hi,

I read that issue, and agree with your assessment; however, the --def-opt flag still does not work as stated on the main page readme:

If you want to get an option argument but fallback on a default value if it doesn't exist then you can use the argagg::option_results::as() API and provide a default value

When the value doesn't exist, I actually get that error described in my opening post. Am I misunderstanding this?

Thanks

from argagg.

I see, the wording in my README is poor then. I think it is more precisely worded as:

If you want to get an option argument but fallback on a default value if the option is not specified then you can use the argagg::option_results::as() API and provide a default value.

For the sample, I added some unit test cases that hopefully clarify the behavior:

• c79fe8a#diff-66a301af68cc98d54d6f489be5a4699aR114
• c79fe8a#diff-66a301af68cc98d54d6f489be5a4699aR154
• c79fe8a#diff-66a301af68cc98d54d6f489be5a4699aR161
• https://travis-ci.org/vietjtnguyen/argagg/builds/373669320

Let me know if that clarifies the behavior or not.

from argagg.

Ah OK, I understand better now. Thanks for the clarification.

from argagg.