diff options
| author | David Bremner <david@tethera.net> | 2017-12-14 22:29:57 -0400 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2017-12-21 09:22:30 -0400 |
| commit | 7cfa1c69610bcf15fb47868131d3d9102f899225 (patch) | |
| tree | 1e86f76b0c8cb10db1aca5763933c334e6846d9d /test | |
| parent | 8e1c3fa5bba1434077df2f5a81ed6588468f9eb9 (diff) | |
lib: return "" rather than NULL from notmuch_thread_get_authors
The current behaviour is at best under-documented. The modified test in
T470-missing-headers.sh previously relied on printf doing the right
thing with NULL, which seems icky.
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.
Diffstat (limited to 'test')
| -rwxr-xr-x | test/T470-missing-headers.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/T470-missing-headers.sh b/test/T470-missing-headers.sh index 4bf5d285..555fd4e9 100755 --- a/test/T470-missing-headers.sh +++ b/test/T470-missing-headers.sh @@ -25,7 +25,7 @@ NOTMUCH_NEW >/dev/null test_begin_subtest "Search: text" output=$(notmuch search '*' | notmuch_search_sanitize) test_expect_equal "$output" "\ -thread:XXX 2001-01-05 [1/1] (null); (inbox unread) +thread:XXX 2001-01-05 [1/1] ; (inbox unread) thread:XXX 1970-01-01 [1/1] Notmuch Test Suite; (inbox unread)" test_begin_subtest "Search: json" |