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.
_______________________________________________
notmuch mailing list
[email protected]
http://notmuchmail.org/mailman/listinfo/notmuch