[PATCH 0/9] argument parsing fixes and improvements

David Bremner david at tethera.net
Sun Oct 1 04:11:09 PDT 2017


Jani Nikula <jani at nikula.org> writes:

> On Sat, 30 Sep 2017, Jani Nikula <jani at nikula.org> wrote:
>> Looking at the defaults from another angle, if we don't want the ability
>> to set --foo=default explicitly, I still think passing ints as booleans
>> to the argument parser and checking if a boolean is neither true nor
>> false is the wrong thing to do. I'd also like to convert to stdbool
>> more. But those should not be a reason to convert essentially boolean
>> arguments to keyword arguments. I think we need a way to have the
>> argument parser tell us if an argument was present or not.
>
> id:20170930213239.15392-1-jani at nikula.org would make it easier to add,
> say, a notmuch_bool_t *present field to notmuch_opt_desc_t that we could
> set whenever we see the option and we want to know the difference.

OK, I've queued that patch. Care to add `present` so we can unblock this
discussion?

d


More information about the notmuch mailing list