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

Róman Joost roman at bromeco.de
Fri Dec 15 13:54:02 PST 2017


Dear David,

On Thu, Dec 14, 2017 at 10:29:57PM -0400, David Bremner wrote:
> The current beheviour is at best underdocumented. The modified test in
> T470-missing-headers.sh previously relied on printf doing the right
> thing with NULL, which seems ick.
> 
> The use of talloc_strdup here is probably overkill, but it avoids
> having to enforce that thread->authors is never mutated outside
> _resolve_thread_authors_string.
> [...]

Thanks a lot for this patch. I think it'll help future authors to be
less surprised :))

Kind Regards,
-- 
Róman Joost
email: roman at bromeco.de
GPG key: 3A765B52
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20171216/d1e5c414/attachment.sig>


More information about the notmuch mailing list