[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

[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 +---

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

2014-11-03 Thread Michal Sojka
From: Jani Nikula j...@nikula.org 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 +---

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

2014-11-03 Thread David Bremner
Michal Sojka sojk...@fel.cvut.cz writes: From: Jani Nikula j...@nikula.org 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

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

2014-11-03 Thread David Bremner
David Bremner da...@tethera.net writes: Michal Sojka sojk...@fel.cvut.cz writes: From: Jani Nikula j...@nikula.org 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