David Edmondson <dme@dme.org> writes: > In addition to the 'authors' attribute of each search result, include > 'authors_matched' and 'authors_non_matched' attributes. Both > attributes are always included and are formatted as a list of > authors. If there are no matching authors, the 'authors_non_matched' > attribute is set to the empty list. > --- It would be nicer if the tests were repaired in the same commit, or minimally marked broken. > static int > do_search_threads (sprinter_t *format, > notmuch_query_t *query, > @@ -152,6 +253,10 @@ do_search_threads (sprinter_t *format, > format->integer (format, total); > format->map_key (format, "authors"); > format->string (format, authors); > + if (_enumerate_authors (format, thread) < 0) { > + fprintf (stderr, "Out of memory\n"); > + return 1; > + } Maybe I'm just blind, but I don't see how enumerate authors ever returns anything other than 0.