[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