--- /dev/null
+Return-Path: <amdragon@mit.edu>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+ by olra.theworths.org (Postfix) with ESMTP id 14337431FAF\r
+ for <notmuch@notmuchmail.org>; Wed, 18 Jul 2012 12:56:06 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.7\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
+ tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+ by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+ with ESMTP id X5JBicJO2Qfx for <notmuch@notmuchmail.org>;\r
+ Wed, 18 Jul 2012 12:56:04 -0700 (PDT)\r
+Received: from dmz-mailsec-scanner-1.mit.edu (DMZ-MAILSEC-SCANNER-1.MIT.EDU\r
+ [18.9.25.12])\r
+ by olra.theworths.org (Postfix) with ESMTP id 9D8DF431FAE\r
+ for <notmuch@notmuchmail.org>; Wed, 18 Jul 2012 12:56:04 -0700 (PDT)\r
+X-AuditID: 1209190c-b7f806d000006b87-56-500714d4dc0b\r
+Received: from mailhub-auth-4.mit.edu ( [18.7.62.39])\r
+ by dmz-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP\r
+ id 95.D3.27527.4D417005; Wed, 18 Jul 2012 15:56:04 -0400 (EDT)\r
+Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
+ by mailhub-auth-4.mit.edu (8.13.8/8.9.2) with ESMTP id q6IJu4Lw024341; \r
+ Wed, 18 Jul 2012 15:56:04 -0400\r
+Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
+ (authenticated bits=0)\r
+ (User authenticated as amdragon@ATHENA.MIT.EDU)\r
+ by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q6IJu2Ii011856\r
+ (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
+ Wed, 18 Jul 2012 15:56:03 -0400 (EDT)\r
+Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
+ (envelope-from <amdragon@mit.edu>)\r
+ id 1SraLq-000165-GP; Wed, 18 Jul 2012 15:56:02 -0400\r
+Date: Wed, 18 Jul 2012 15:56:02 -0400\r
+From: Austin Clements <amdragon@MIT.EDU>\r
+To: craven@gmx.net\r
+Subject: Re: [PATCH v6 3/3] Use the structured formatters in notmuch-search.c.\r
+Message-ID: <20120718195602.GQ31670@mit.edu>\r
+References: <20120714020954.GD31670@mit.edu>\r
+ <1342427702-23316-1-git-send-email-craven@gmx.net>\r
+ <1342427702-23316-4-git-send-email-craven@gmx.net>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=iso-8859-1\r
+Content-Disposition: inline\r
+Content-Transfer-Encoding: 8bit\r
+In-Reply-To: <1342427702-23316-4-git-send-email-craven@gmx.net>\r
+User-Agent: Mutt/1.5.21 (2010-09-15)\r
+X-Brightmail-Tracker:\r
+ H4sIAAAAAAAAA+NgFprDKsWRmVeSWpSXmKPExsUixG6nrntFhD3A4P8fHou9De2MFtdvzmR2\r
+ YPJYvGk/m8ezVbeYA5iiuGxSUnMyy1KL9O0SuDK2n1rLXLC5hbGi4f95pgbG98ldjJwcEgIm\r
+ Ehf/rmCEsMUkLtxbzwZiCwnsY5T49Ly0i5ELyN7AKHGpaykzhHOSSWJH12x2CGcJo8TuW5PZ\r
+ QVpYBFQlHp7dAmazCWhIbNu/HGysiICQxKQvr1hAbGYBaYlvv5uZQGxhAT+JnqWHwdbxCuhI\r
+ rDm/FGroNEaJ78++s0IkBCVOznwC1awjsXPrHaAGDrBBy/9xQITlJZq3zmYGCXMK2Es83KED\r
+ EhYVUJGYcnIb2wRG4VlIBs1CMmgWwqBZSAYtYGRZxSibklulm5uYmVOcmqxbnJyYl5dapGuo\r
+ l5tZopeaUrqJERQJnJI8OxjfHFQ6xCjAwajEw/tgF2uAEGtiWXFl7iFGSQ4mJVHeT0LsAUJ8\r
+ SfkplRmJxRnxRaU5qcWHGCU4mJVEeB8IAuV4UxIrq1KL8mFS0hwsSuK8l1Nu+gsJpCeWpGan\r
+ phakFsFkZTg4lCR4k4ERLyRYlJqeWpGWmVOCkGbi4AQZzgM0PAqkhre4IDG3ODMdIn+KUVFK\r
+ nLcYJCEAksgozYPrhSWqV4ziQK8I88qBVPEAkxxc9yugwUxAg7mL2UAGlyQipKQaGMvM16pl\r
+ L7BmvKvluKPw2XutOxs+2b26fmXCmqbVOQ39CafK//lMKZrc7xH4yb58Zfn2J/VnBE3uyz6f\r
+ v9wx683CZ8VywbZSb2fvilhx/cDyhwdlpaYrsDh9z2g6yRp1abb465q7XQ/nJPQxB/2eeErT\r
+ fuf2Rr/Tn0VnOOy3cbQROW5bljnvGb8SS3FGoqEWc1FxIgA3t36uLwMAAA==\r
+Cc: notmuch@notmuchmail.org\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+ <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Wed, 18 Jul 2012 19:56:06 -0000\r
+\r
+Just a few comments (don't forget to scroll all the way down).\r
+Overall this is looking pretty good.\r
+\r
+Quoth craven@gmx.net on Jul 16 at 10:35 am:\r
+> This patch switches from the current ad-hoc printer to the structured\r
+> formatters in sprinter.h, sprinter-text-search.c and sprinter-json.c.\r
+> \r
+> The JSON tests are changed slightly in order to make them PASS for the\r
+> new structured output formatter.\r
+> \r
+> The text tests pass without adaptation.\r
+> ---\r
+> notmuch-search.c | 300 ++++++++++++++++---------------------------------------\r
+> test/json | 18 +---\r
+> 2 files changed, 86 insertions(+), 232 deletions(-)\r
+\r
+That's a fantastic diffstat.\r
+\r
+> \r
+> diff --git a/notmuch-search.c b/notmuch-search.c\r
+> index 3be296d..cf927e6 100644\r
+> --- a/notmuch-search.c\r
+> +++ b/notmuch-search.c\r
+> @@ -19,6 +19,7 @@\r
+> */\r
+> \r
+> #include "notmuch-client.h"\r
+> +#include "sprinter.h"\r
+> \r
+> typedef enum {\r
+> OUTPUT_SUMMARY,\r
+> @@ -28,92 +29,6 @@ typedef enum {\r
+> OUTPUT_TAGS\r
+> } output_t;\r
+> \r
+> -typedef struct search_format {\r
+> - const char *results_start;\r
+> - const char *item_start;\r
+> - void (*item_id) (const void *ctx,\r
+> - const char *item_type,\r
+> - const char *item_id);\r
+> - void (*thread_summary) (const void *ctx,\r
+> - const char *thread_id,\r
+> - const time_t date,\r
+> - const int matched,\r
+> - const int total,\r
+> - const char *authors,\r
+> - const char *subject);\r
+> - const char *tag_start;\r
+> - const char *tag;\r
+> - const char *tag_sep;\r
+> - const char *tag_end;\r
+> - const char *item_sep;\r
+> - const char *item_end;\r
+> - const char *results_end;\r
+> - const char *results_null;\r
+> -} search_format_t;\r
+> -\r
+> -static void\r
+> -format_item_id_text (const void *ctx,\r
+> - const char *item_type,\r
+> - const char *item_id);\r
+> -\r
+> -static void\r
+> -format_thread_text (const void *ctx,\r
+> - const char *thread_id,\r
+> - const time_t date,\r
+> - const int matched,\r
+> - const int total,\r
+> - const char *authors,\r
+> - const char *subject);\r
+> -static const search_format_t format_text = {\r
+> - "",\r
+> - "",\r
+> - format_item_id_text,\r
+> - format_thread_text,\r
+> - " (",\r
+> - "%s", " ",\r
+> - ")", "\n",\r
+> - "",\r
+> - "\n",\r
+> - "",\r
+> -};\r
+> -\r
+> -static void\r
+> -format_item_id_json (const void *ctx,\r
+> - const char *item_type,\r
+> - const char *item_id);\r
+> -\r
+> -static void\r
+> -format_thread_json (const void *ctx,\r
+> - const char *thread_id,\r
+> - const time_t date,\r
+> - const int matched,\r
+> - const int total,\r
+> - const char *authors,\r
+> - const char *subject);\r
+> -\r
+> -/* Any changes to the JSON format should be reflected in the file\r
+> - * devel/schemata. */\r
+> -static const search_format_t format_json = {\r
+> - "[",\r
+> - "{",\r
+> - format_item_id_json,\r
+> - format_thread_json,\r
+> - "\"tags\": [",\r
+> - "\"%s\"", ", ",\r
+> - "]", ",\n",\r
+> - "}",\r
+> - "]\n",\r
+> - "]\n",\r
+> -};\r
+> -\r
+> -static void\r
+> -format_item_id_text (unused (const void *ctx),\r
+> - const char *item_type,\r
+> - const char *item_id)\r
+> -{\r
+> - printf ("%s%s", item_type, item_id);\r
+> -}\r
+> -\r
+> static char *\r
+> sanitize_string (const void *ctx, const char *str)\r
+> {\r
+> @@ -131,72 +46,8 @@ sanitize_string (const void *ctx, const char *str)\r
+> return out;\r
+> }\r
+> \r
+> -static void\r
+> -format_thread_text (const void *ctx,\r
+> - const char *thread_id,\r
+> - const time_t date,\r
+> - const int matched,\r
+> - const int total,\r
+> - const char *authors,\r
+> - const char *subject)\r
+> -{\r
+> - void *ctx_quote = talloc_new (ctx);\r
+> -\r
+> - printf ("thread:%s %12s [%d/%d] %s; %s",\r
+> - thread_id,\r
+> - notmuch_time_relative_date (ctx, date),\r
+> - matched,\r
+> - total,\r
+> - sanitize_string (ctx_quote, authors),\r
+> - sanitize_string (ctx_quote, subject));\r
+> -\r
+> - talloc_free (ctx_quote);\r
+> -}\r
+> -\r
+> -static void\r
+> -format_item_id_json (const void *ctx,\r
+> - unused (const char *item_type),\r
+> - const char *item_id)\r
+> -{\r
+> - void *ctx_quote = talloc_new (ctx);\r
+> -\r
+> - printf ("%s", json_quote_str (ctx_quote, item_id));\r
+> -\r
+> - talloc_free (ctx_quote);\r
+> - \r
+> -}\r
+> -\r
+> -static void\r
+> -format_thread_json (const void *ctx,\r
+> - const char *thread_id,\r
+> - const time_t date,\r
+> - const int matched,\r
+> - const int total,\r
+> - const char *authors,\r
+> - const char *subject)\r
+> -{\r
+> - void *ctx_quote = talloc_new (ctx);\r
+> -\r
+> - printf ("\"thread\": %s,\n"\r
+> - "\"timestamp\": %ld,\n"\r
+> - "\"date_relative\": \"%s\",\n"\r
+> - "\"matched\": %d,\n"\r
+> - "\"total\": %d,\n"\r
+> - "\"authors\": %s,\n"\r
+> - "\"subject\": %s,\n",\r
+> - json_quote_str (ctx_quote, thread_id),\r
+> - date,\r
+> - notmuch_time_relative_date (ctx, date),\r
+> - matched,\r
+> - total,\r
+> - json_quote_str (ctx_quote, authors),\r
+> - json_quote_str (ctx_quote, subject));\r
+> -\r
+> - talloc_free (ctx_quote);\r
+> -}\r
+> -\r
+> static int\r
+> -do_search_threads (const search_format_t *format,\r
+> +do_search_threads (sprinter_t *format,\r
+> notmuch_query_t *query,\r
+> notmuch_sort_t sort,\r
+> output_t output,\r
+> @@ -207,7 +58,6 @@ do_search_threads (const search_format_t *format,\r
+> notmuch_threads_t *threads;\r
+> notmuch_tags_t *tags;\r
+> time_t date;\r
+> - int first_thread = 1;\r
+> int i;\r
+> \r
+> if (offset < 0) {\r
+> @@ -220,14 +70,12 @@ do_search_threads (const search_format_t *format,\r
+> if (threads == NULL)\r
+> return 1;\r
+> \r
+> - fputs (format->results_start, stdout);\r
+> + format->begin_list (format);\r
+> \r
+> for (i = 0;\r
+> notmuch_threads_valid (threads) && (limit < 0 || i < offset + limit);\r
+> notmuch_threads_move_to_next (threads), i++)\r
+> {\r
+> - int first_tag = 1;\r
+> -\r
+> thread = notmuch_threads_get (threads);\r
+> \r
+> if (i < offset) {\r
+> @@ -235,60 +83,96 @@ do_search_threads (const search_format_t *format,\r
+> continue;\r
+> }\r
+> \r
+> - if (! first_thread)\r
+> - fputs (format->item_sep, stdout);\r
+> -\r
+> if (output == OUTPUT_THREADS) {\r
+> - format->item_id (thread, "thread:",\r
+> - notmuch_thread_get_thread_id (thread));\r
+> + format->set_prefix (format, "thread");\r
+> + format->string (format,\r
+> + notmuch_thread_get_thread_id (thread));\r
+> + format->separator (format);\r
+> } else { /* output == OUTPUT_SUMMARY */\r
+> - fputs (format->item_start, stdout);\r
+> + void *ctx_quote = talloc_new (thread);\r
+> + const char *authors = notmuch_thread_get_authors (thread);\r
+> + const char *subject = notmuch_thread_get_subject (thread);\r
+> + const char *thread_id = notmuch_thread_get_thread_id (thread);\r
+> + int matched = notmuch_thread_get_matched_messages (thread);\r
+> + int total = notmuch_thread_get_total_messages (thread);\r
+> + const char *relative_date = NULL;\r
+> + notmuch_bool_t first_tag = TRUE;\r
+> +\r
+> + format->begin_map (format);\r
+> \r
+> if (sort == NOTMUCH_SORT_OLDEST_FIRST)\r
+> date = notmuch_thread_get_oldest_date (thread);\r
+> else\r
+> date = notmuch_thread_get_newest_date (thread);\r
+> \r
+> - format->thread_summary (thread,\r
+> - notmuch_thread_get_thread_id (thread),\r
+> - date,\r
+> - notmuch_thread_get_matched_messages (thread),\r
+> - notmuch_thread_get_total_messages (thread),\r
+> - notmuch_thread_get_authors (thread),\r
+> - notmuch_thread_get_subject (thread));\r
+> + relative_date = notmuch_time_relative_date (ctx_quote, date);\r
+> +\r
+> + if (format->is_text_printer (format)) {\r
+> + /* Special case for the text formatter */\r
+> + printf ("thread:%s %12s [%d/%d] %s; %s (",\r
+> + thread_id,\r
+> + relative_date,\r
+> + matched,\r
+> + total,\r
+> + sanitize_string (ctx_quote, authors),\r
+> + sanitize_string (ctx_quote, subject));\r
+\r
+Great. This seems much simpler than trying to track the context in\r
+the text printer.\r
+\r
+> + } else { /* Structured Output */\r
+> + format->map_key (format, "thread");\r
+> + format->string (format, thread_id);\r
+> + format->map_key (format, "timestamp");\r
+> + format->integer (format, date);\r
+> + format->map_key (format, "date_relative");\r
+> + format->string (format, relative_date);\r
+> + format->map_key (format, "matched");\r
+> + format->integer (format, matched);\r
+> + format->map_key (format, "total");\r
+> + format->integer (format, total);\r
+> + format->map_key (format, "authors");\r
+> + format->string (format, authors);\r
+> + format->map_key (format, "subject");\r
+> + format->string (format, subject);\r
+> + }\r
+> +\r
+> + talloc_free (ctx_quote);\r
+> \r
+> - fputs (format->tag_start, stdout);\r
+> + format->map_key (format, "tags");\r
+> + format->begin_list (format);\r
+> \r
+> for (tags = notmuch_thread_get_tags (thread);\r
+> notmuch_tags_valid (tags);\r
+> notmuch_tags_move_to_next (tags))\r
+> {\r
+> - if (! first_tag)\r
+> - fputs (format->tag_sep, stdout);\r
+> - printf (format->tag, notmuch_tags_get (tags));\r
+> - first_tag = 0;\r
+> + const char *tag = notmuch_tags_get (tags);\r
+> +\r
+> + if (format->is_text_printer (format)) {\r
+> + /* Special case for the text formatter */\r
+> + if (first_tag)\r
+> + first_tag = FALSE;\r
+> + else\r
+> + fputc (' ', stdout);\r
+> + fputs (tag, stdout);\r
+> + } else /* Structured Output */\r
+\r
+Please put braces around the else part as well (since you need braces\r
+around the if part).\r
+\r
+> + format->string (format, tag);\r
+> }\r
+> \r
+> - fputs (format->tag_end, stdout);\r
+> + if (format->is_text_printer (format))\r
+> + printf (")");\r
+> \r
+> - fputs (format->item_end, stdout);\r
+> + format->end (format);\r
+> + format->end (format);\r
+> + format->separator (format);\r
+> }\r
+> \r
+> - first_thread = 0;\r
+> -\r
+> notmuch_thread_destroy (thread);\r
+> }\r
+> \r
+> - if (first_thread)\r
+> - fputs (format->results_null, stdout);\r
+> - else\r
+> - fputs (format->results_end, stdout);\r
+> + format->end (format);\r
+> \r
+> return 0;\r
+> }\r
+> \r
+> static int\r
+> -do_search_messages (const search_format_t *format,\r
+> +do_search_messages (sprinter_t *format,\r
+> notmuch_query_t *query,\r
+> output_t output,\r
+> int offset,\r
+> @@ -297,7 +181,6 @@ do_search_messages (const search_format_t *format,\r
+> notmuch_message_t *message;\r
+> notmuch_messages_t *messages;\r
+> notmuch_filenames_t *filenames;\r
+> - int first_message = 1;\r
+> int i;\r
+> \r
+> if (offset < 0) {\r
+> @@ -310,7 +193,7 @@ do_search_messages (const search_format_t *format,\r
+> if (messages == NULL)\r
+> return 1;\r
+> \r
+> - fputs (format->results_start, stdout);\r
+> + format->begin_list (format);\r
+> \r
+> for (i = 0;\r
+> notmuch_messages_valid (messages) && (limit < 0 || i < offset + limit);\r
+> @@ -328,24 +211,17 @@ do_search_messages (const search_format_t *format,\r
+> notmuch_filenames_valid (filenames);\r
+> notmuch_filenames_move_to_next (filenames))\r
+> {\r
+> - if (! first_message)\r
+> - fputs (format->item_sep, stdout);\r
+> -\r
+> - format->item_id (message, "",\r
+> - notmuch_filenames_get (filenames));\r
+> -\r
+> - first_message = 0;\r
+> + format->string (format, notmuch_filenames_get (filenames));\r
+> + format->separator (format);\r
+> }\r
+> \r
+> notmuch_filenames_destroy( filenames );\r
+> \r
+> } else { /* output == OUTPUT_MESSAGES */\r
+> - if (! first_message)\r
+> - fputs (format->item_sep, stdout);\r
+> -\r
+> - format->item_id (message, "id:",\r
+> - notmuch_message_get_message_id (message));\r
+> - first_message = 0;\r
+> + format->set_prefix (format, "id");\r
+> + format->string (format,\r
+> + notmuch_message_get_message_id (message));\r
+> + format->separator (format);\r
+> }\r
+> \r
+> notmuch_message_destroy (message);\r
+> @@ -353,23 +229,19 @@ do_search_messages (const search_format_t *format,\r
+> \r
+> notmuch_messages_destroy (messages);\r
+> \r
+> - if (first_message)\r
+> - fputs (format->results_null, stdout);\r
+> - else\r
+> - fputs (format->results_end, stdout);\r
+> + format->end (format);\r
+> \r
+> return 0;\r
+> }\r
+> \r
+> static int\r
+> do_search_tags (notmuch_database_t *notmuch,\r
+> - const search_format_t *format,\r
+> + sprinter_t *format,\r
+> notmuch_query_t *query)\r
+> {\r
+> notmuch_messages_t *messages = NULL;\r
+> notmuch_tags_t *tags;\r
+> const char *tag;\r
+> - int first_tag = 1;\r
+> \r
+> /* should the following only special case if no excluded terms\r
+> * specified? */\r
+> @@ -387,7 +259,7 @@ do_search_tags (notmuch_database_t *notmuch,\r
+> if (tags == NULL)\r
+> return 1;\r
+> \r
+> - fputs (format->results_start, stdout);\r
+> + format->begin_list (format);\r
+> \r
+> for (;\r
+> notmuch_tags_valid (tags);\r
+> @@ -395,12 +267,9 @@ do_search_tags (notmuch_database_t *notmuch,\r
+> {\r
+> tag = notmuch_tags_get (tags);\r
+> \r
+> - if (! first_tag)\r
+> - fputs (format->item_sep, stdout);\r
+> + format->string (format, tag);\r
+> + format->separator (format);\r
+> \r
+> - format->item_id (tags, "", tag);\r
+> -\r
+> - first_tag = 0;\r
+> }\r
+> \r
+> notmuch_tags_destroy (tags);\r
+> @@ -408,10 +277,7 @@ do_search_tags (notmuch_database_t *notmuch,\r
+> if (messages)\r
+> notmuch_messages_destroy (messages);\r
+> \r
+> - if (first_tag)\r
+> - fputs (format->results_null, stdout);\r
+> - else\r
+> - fputs (format->results_end, stdout);\r
+> + format->end (format);\r
+> \r
+> return 0;\r
+> }\r
+> @@ -430,7 +296,7 @@ notmuch_search_command (void *ctx, int argc, char *argv[])\r
+> notmuch_query_t *query;\r
+> char *query_str;\r
+> notmuch_sort_t sort = NOTMUCH_SORT_NEWEST_FIRST;\r
+> - const search_format_t *format = &format_text;\r
+> + sprinter_t *format = NULL;\r
+> int opt_index, ret;\r
+> output_t output = OUTPUT_SUMMARY;\r
+> int offset = 0;\r
+> @@ -475,10 +341,10 @@ notmuch_search_command (void *ctx, int argc, char *argv[])\r
+> \r
+> switch (format_sel) {\r
+> case NOTMUCH_FORMAT_TEXT:\r
+> - format = &format_text;\r
+> + format = sprinter_text_search_create (ctx, stdout);\r
+> break;\r
+> case NOTMUCH_FORMAT_JSON:\r
+> - format = &format_json;\r
+> + format = sprinter_json_create (ctx, stdout);\r
+> break;\r
+> }\r
+> \r
+> @@ -546,5 +412,7 @@ notmuch_search_command (void *ctx, int argc, char *argv[])\r
+> notmuch_query_destroy (query);\r
+> notmuch_database_destroy (notmuch);\r
+> \r
+> + talloc_free (format);\r
+> +\r
+> return ret;\r
+> }\r
+> diff --git a/test/json b/test/json\r
+> index 6439788..f0ebf08 100755\r
+> --- a/test/json\r
+> +++ b/test/json\r
+> @@ -10,14 +10,7 @@ test_expect_equal "$output" "[[[{\"id\": \"${gen_msg_id}\", \"match\": true, \"e\r
+> test_begin_subtest "Search message: json"\r
+> add_message "[subject]=\"json-search-subject\"" "[date]=\"Sat, 01 Jan 2000 12:00:00 -0000\"" "[body]=\"json-search-message\""\r
+> output=$(notmuch search --format=json "json-search-message" | notmuch_search_sanitize)\r
+\r
+What's your reason for not piping this through\r
+notmuch_json_show_sanitize?\r
+\r
+> -test_expect_equal "$output" "[{\"thread\": \"XXX\",\r
+> -\"timestamp\": 946728000,\r
+> -\"date_relative\": \"2000-01-01\",\r
+> -\"matched\": 1,\r
+> -\"total\": 1,\r
+> -\"authors\": \"Notmuch Test Suite\",\r
+> -\"subject\": \"json-search-subject\",\r
+> -\"tags\": [\"inbox\", \"unread\"]}]"\r
+> +test_expect_equal "$output" "[{\"thread\": \"XXX\", \"timestamp\": 946728000, \"date_relative\": \"2000-01-01\", \"matched\": 1, \"total\": 1, \"authors\": \"Notmuch Test Suite\", \"subject\": \"json-search-subject\", \"tags\": [\"inbox\", \"unread\"]}]"\r
+> \r
+> test_begin_subtest "Show message: json, utf-8"\r
+> add_message "[subject]=\"json-show-utf8-body-sübjéct\"" "[date]=\"Sat, 01 Jan 2000 12:00:00 -0000\"" "[body]=\"jsön-show-méssage\""\r
+> @@ -40,13 +33,6 @@ test_expect_equal "$output" "[[[{\"id\": \"$id\", \"match\": true, \"excluded\":\r
+> test_begin_subtest "Search message: json, utf-8"\r
+> add_message "[subject]=\"json-search-utf8-body-sübjéct\"" "[date]=\"Sat, 01 Jan 2000 12:00:00 -0000\"" "[body]=\"jsön-search-méssage\""\r
+> output=$(notmuch search --format=json "jsön-search-méssage" | notmuch_search_sanitize)\r
+> -test_expect_equal "$output" "[{\"thread\": \"XXX\",\r
+> -\"timestamp\": 946728000,\r
+> -\"date_relative\": \"2000-01-01\",\r
+> -\"matched\": 1,\r
+> -\"total\": 1,\r
+> -\"authors\": \"Notmuch Test Suite\",\r
+> -\"subject\": \"json-search-utf8-body-sübjéct\",\r
+> -\"tags\": [\"inbox\", \"unread\"]}]"\r
+> +test_expect_equal "$output" "[{\"thread\": \"XXX\", \"timestamp\": 946728000, \"date_relative\": \"2000-01-01\", \"matched\": 1, \"total\": 1, \"authors\": \"Notmuch Test Suite\", \"subject\": \"json-search-utf8-body-sübjéct\", \"tags\": [\"inbox\", \"unread\"]}]"\r
+> \r
+> test_done\r