On Sun, 30 Aug 2015, David Bremner <[email protected]> wrote: > Jani Nikula <[email protected]> writes: > > >> + { NOTMUCH_OPT_KEYWORD, &ctx->dupe, "deduplicate", 'x', > > probably you want 'D' or 'd' here. Not that it makes a practical > difference at this point. > >> + (notmuch_keyword_t []){ { "yes", -1 }, > > I'm not very enthusiastic about reusing ctx->dupe for this. > In particular the use of -1 for yes is off-putting > It seems better to allocate another word of memory and use an > enum, as elsewhere.
Agreed on both. BR, Jani. _______________________________________________ notmuch mailing list [email protected] http://notmuchmail.org/mailman/listinfo/notmuch
