Re: [PATCH] cli: add --output=address-{from,to,all} to notmuch search

2014-09-19 Thread David Bremner
Jani Nikula j...@nikula.org writes: + if (name *name) + full_address = talloc_asprintf (NULL, %s %s, name, addr); + else + full_address = talloc_asprintf (NULL, %s, addr Is there some reason not to use sprinter as a talloc context here? + +

Re: [PATCH] cli: add --output=address-{from, to, all} to notmuch search

2014-09-06 Thread Jani Nikula
On Sat, 06 Sep 2014, Jani Nikula j...@nikula.org wrote: address-from prints reply-to or from, address-to prints to, cc, and bcc, and address-all prints all of them. *sigh* this was supposed to be: --- cli: add --output=sender and --output=recipients to notmuch search sender prints from,

Re: [PATCH] cli: add --output=address-{from, to, all} to notmuch search

2014-09-06 Thread Mark Walters
On Sat, 06 Sep 2014, Jani Nikula j...@nikula.org wrote: address-from prints reply-to or from, address-to prints to, cc, and bcc, and address-all prints all of them. This looks good to me. Obviously needs the new commit message. I think we should think about the deduplication possibilities