Re: [PATCH] lib: return "" rather than NULL from notmuch_thread_get_authors

2017-12-20 Thread Tomi Ollila
On Thu, Dec 14 2017, David Bremner wrote: > The current beheviour is at best underdocumented. The modified test in Typo in commit message ;), otherwise looks reasonable to me > T470-missing-headers.sh previously relied on printf doing the right > thing with NULL, which seems ick. > > The use of

Re: [PATCH] cli: add support for only printing the addresses in notmuch address

2017-12-20 Thread Jameson Graef Rollins
On Wed, Dec 20 2017, Jani Nikula wrote: > Matter of taste. I like the self-documenting aspect of *what* these > options control. --output=body is obvious, --body is less > so. --include-html includes html somewhere, I think --output=html would > be better. > > In the spirit of

Re: [PATCH] cli: add support for only printing the addresses in notmuch address

2017-12-20 Thread Jani Nikula
On Tue, 19 Dec 2017, Jameson Graef Rollins wrote: > On Wed, Dec 20 2017, Jani Nikula wrote: >> ~$ notmuch address --output=sender --output=recipients --output=address >> --output=count id:878tdy8a2q@ligo.caltech.edu >> 1