Re: [PATCH 2/2] cli: convert "notmuch new" to the argument parser

Subject: Re: [PATCH 2/2] cli: convert "notmuch new" to the argument parser

Date: Tue, 04 Dec 2012 00:35:46 +0200

To: Austin Clements

Cc: notmuch@notmuchmail.org

From: Jani Nikula


On Tue, 04 Dec 2012, Austin Clements <amdragon@MIT.EDU> wrote:
>> -    argc--; argv++; /* skip subcommand argument */
>> +    notmuch_opt_desc_t options[] = {
>> +	{ NOTMUCH_OPT_BOOLEAN,  &add_files_state.verbose, "verbose", 'v', 0 },
>> +	{ NOTMUCH_OPT_BOOLEAN,  &add_files_state.debug, "debug", 'd', 0 },
>> +	{ NOTMUCH_OPT_BOOLEAN,  &no_hooks, "no-hooks", 'n', 0 },
>
> Not that the single character arguments matter right now, but '-n' is
> often used for "dry-run", which could actually apply to notmuch new.

Agreed, although I believe we'd have to review all of the single
character arguments before actually enabling any of them. At least my
choice of them has been rather whimsical. I'll leave this up to David to
decide.

BR,
Jani.

Thread: