[PATCH v2 05/10] cli: add support for hierarchical command line option arrays

2014-11-04 Thread David Bremner
David Bremner writes: > Michal Sojka writes: > >> From: Jani Nikula >> >> NOTMUCH_OPT_INHERIT expects a notmuch_opt_desc_t * pointer in >> output_var. > > At the risk of bikeshedding, what about NOTMUCH_OPT_RECURSE instead of > _INHERIT? Hrm. Now that I read the next patch, I retract my sugges

[PATCH v2 05/10] cli: add support for hierarchical command line option arrays

2014-11-04 Thread David Bremner
Michal Sojka writes: > From: Jani Nikula > > NOTMUCH_OPT_INHERIT expects a notmuch_opt_desc_t * pointer in > output_var. At the risk of bikeshedding, what about NOTMUCH_OPT_RECURSE instead of _INHERIT?

[PATCH v2 05/10] cli: add support for hierarchical command line option arrays

2014-11-04 Thread Michal Sojka
From: Jani Nikula NOTMUCH_OPT_INHERIT expects a notmuch_opt_desc_t * pointer in output_var. The "Unrecognized option" message was moved out of parse_option() to not be emitted twice or when parsing a non-inherited option. --- command-line-arguments.c | 16 +--- command-line-argument

Re: [PATCH v2 05/10] cli: add support for hierarchical command line option arrays

2014-11-03 Thread David Bremner
David Bremner writes: > Michal Sojka writes: > >> From: Jani Nikula >> >> NOTMUCH_OPT_INHERIT expects a notmuch_opt_desc_t * pointer in >> output_var. > > At the risk of bikeshedding, what about NOTMUCH_OPT_RECURSE instead of > _INHERIT? Hrm. Now that I read the next patch, I retract my sugges

Re: [PATCH v2 05/10] cli: add support for hierarchical command line option arrays

2014-11-03 Thread David Bremner
Michal Sojka writes: > From: Jani Nikula > > NOTMUCH_OPT_INHERIT expects a notmuch_opt_desc_t * pointer in > output_var. At the risk of bikeshedding, what about NOTMUCH_OPT_RECURSE instead of _INHERIT? ___ notmuch mailing list notmuch@notmuchmail.org

[PATCH v2 05/10] cli: add support for hierarchical command line option arrays

2014-11-03 Thread Michal Sojka
From: Jani Nikula NOTMUCH_OPT_INHERIT expects a notmuch_opt_desc_t * pointer in output_var. The "Unrecognized option" message was moved out of parse_option() to not be emitted twice or when parsing a non-inherited option. --- command-line-arguments.c | 16 +--- command-line-argument