Re: [PATCH v7 3/3] Use the structured formatters in notmuch-search.c.
authorTomi Ollila <tomi.ollila@iki.fi>
Fri, 20 Jul 2012 09:14:17 +0000 (12:14 +0300)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:48:25 +0000 (09:48 -0800)
04/1b3d5954a04ae65b12a5cb49d28928663a0a7c [new file with mode: 0644]

diff --git a/04/1b3d5954a04ae65b12a5cb49d28928663a0a7c b/04/1b3d5954a04ae65b12a5cb49d28928663a0a7c
new file mode 100644 (file)
index 0000000..5c52782
--- /dev/null
@@ -0,0 +1,621 @@
+Return-Path: <tomi.ollila@iki.fi>\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 DE2CC431FBC\r
+       for <notmuch@notmuchmail.org>; Fri, 20 Jul 2012 02:14:08 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: 0\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
+       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 5kNdJfN71GQs for <notmuch@notmuchmail.org>;\r
+       Fri, 20 Jul 2012 02:14:07 -0700 (PDT)\r
+Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
+       by olra.theworths.org (Postfix) with ESMTP id 63AA0431FAE\r
+       for <notmuch@notmuchmail.org>; Fri, 20 Jul 2012 02:14:07 -0700 (PDT)\r
+Received: by guru.guru-group.fi (Postfix, from userid 501)\r
+       id 48B7010014D; Fri, 20 Jul 2012 12:14:18 +0300 (EEST)\r
+From: Tomi Ollila <tomi.ollila@iki.fi>\r
+To: craven@gmx.net, notmuch@notmuchmail.org\r
+Subject: Re: [PATCH v7 3/3] Use the structured formatters in notmuch-search.c.\r
+In-Reply-To: <1342766173-1344-4-git-send-email-craven@gmx.net>\r
+References: <20120718194819.GP31670@mit.edu>\r
+       <1342766173-1344-1-git-send-email-craven@gmx.net>\r
+       <1342766173-1344-4-git-send-email-craven@gmx.net>\r
+User-Agent: Notmuch/0.13.2+93~ge4fdd97 (http://notmuchmail.org) Emacs/23.1.1\r
+       (x86_64-redhat-linux-gnu)\r
+X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
+       $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
+       !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
+Date: Fri, 20 Jul 2012 12:14:17 +0300\r
+Message-ID: <m2r4s694ly.fsf@guru.guru-group.fi>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=utf-8\r
+Content-Transfer-Encoding: quoted-printable\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: Fri, 20 Jul 2012 09:14:09 -0000\r
+\r
+On Fri, Jul 20 2012, craven@gmx.net wrote:\r
+\r
+> From: <craven@gmx.net>\r
+>\r
+> This patch switches from the current ad-hoc printer to the structured\r
+> formatters in sprinter.h, sprinter-text.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 | 301 ++++++++++++++++---------------------------------=\r
+------\r
+>  test/json        |  34 ++++---\r
+>  2 files changed, 103 insertions(+), 232 deletions(-)\r
+>\r
+> diff --git a/notmuch-search.c b/notmuch-search.c\r
+> index 3be296d..07211e8 100644\r
+> --- a/notmuch-search.c\r
+> +++ b/notmuch-search.c\r
+> @@ -19,6 +19,7 @@\r
+>   */\r
+>=20=20\r
+>  #include "notmuch-client.h"\r
+> +#include "sprinter.h"\r
+>=20=20\r
+>  typedef enum {\r
+>      OUTPUT_SUMMARY,\r
+> @@ -28,92 +29,6 @@ typedef enum {\r
+>      OUTPUT_TAGS\r
+>  } output_t;\r
+>=20=20\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 =3D {\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 =3D {\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
+>=20=20\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 =3D 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 =3D talloc_new (ctx);\r
+> -\r
+> -    printf ("%s", json_quote_str (ctx_quote, item_id));\r
+> -\r
+> -    talloc_free (ctx_quote);\r
+> -=20=20=20=20\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 =3D 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 =3D 1;\r
+>      int i;\r
+>=20=20\r
+>      if (offset < 0) {\r
+> @@ -220,14 +70,12 @@ do_search_threads (const search_format_t *format,\r
+>      if (threads =3D=3D NULL)\r
+>      return 1;\r
+>=20=20\r
+> -    fputs (format->results_start, stdout);\r
+> +    format->begin_list (format);\r
+>=20=20\r
+>      for (i =3D 0;\r
+>       notmuch_threads_valid (threads) && (limit < 0 || i < offset + limit);\r
+>       notmuch_threads_move_to_next (threads), i++)\r
+>      {\r
+> -    int first_tag =3D 1;\r
+> -\r
+>      thread =3D notmuch_threads_get (threads);\r
+>=20=20\r
+>      if (i < offset) {\r
+> @@ -235,60 +83,97 @@ do_search_threads (const search_format_t *format,\r
+>          continue;\r
+>      }\r
+>=20=20\r
+> -    if (! first_thread)\r
+> -        fputs (format->item_sep, stdout);\r
+> -\r
+>      if (output =3D=3D 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 =3D=3D OUTPUT_SUMMARY */\r
+> -        fputs (format->item_start, stdout);\r
+> +        void *ctx_quote =3D talloc_new (thread);\r
+> +        const char *authors =3D notmuch_thread_get_authors (thread);\r
+> +        const char *subject =3D notmuch_thread_get_subject (thread);\r
+> +        const char *thread_id =3D notmuch_thread_get_thread_id (thread);\r
+> +        int matched =3D notmuch_thread_get_matched_messages (thread);\r
+> +        int total =3D notmuch_thread_get_total_messages (thread);\r
+> +        const char *relative_date =3D NULL;\r
+> +        notmuch_bool_t first_tag =3D TRUE;\r
+> +\r
+> +        format->begin_map (format);\r
+>=20=20\r
+>          if (sort =3D=3D NOTMUCH_SORT_OLDEST_FIRST)\r
+>              date =3D notmuch_thread_get_oldest_date (thread);\r
+>          else\r
+>              date =3D notmuch_thread_get_newest_date (thread);\r
+>=20=20\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 =3D notmuch_time_relative_date (ctx_quote, date);\r
+> +\r
+> +        if (format->is_text_printer) {\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
+> +        } 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
+>=20=20\r
+> -        fputs (format->tag_start, stdout);\r
+> +        format->map_key (format, "tags");\r
+> +        format->begin_list (format);\r
+>=20=20\r
+>          for (tags =3D 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 =3D 0;\r
+> +            const char *tag =3D notmuch_tags_get (tags);\r
+> +\r
+> +            if (format->is_text_printer) {\r
+> +                  /* Special case for the text formatter */\r
+> +                if (first_tag)\r
+> +                    first_tag =3D FALSE;\r
+> +                else\r
+> +                    fputc (' ', stdout);\r
+> +                fputs (tag, stdout);\r
+> +            } else { /* Structured Output */\r
+> +                format->string (format, tag);\r
+> +            }\r
+>          }\r
+>=20=20\r
+> -        fputs (format->tag_end, stdout);\r
+> +        if (format->is_text_printer)\r
+> +            printf (")");\r
+>=20=20\r
+> -        fputs (format->item_end, stdout);\r
+> +        format->end (format);\r
+> +        format->end (format);\r
+> +        format->separator (format);\r
+>      }\r
+>=20=20\r
+> -    first_thread =3D 0;\r
+> -\r
+>      notmuch_thread_destroy (thread);\r
+>      }\r
+>=20=20\r
+> -    if (first_thread)\r
+> -    fputs (format->results_null, stdout);\r
+> -    else\r
+> -    fputs (format->results_end, stdout);\r
+> +    format->end (format);\r
+>=20=20\r
+>      return 0;\r
+>  }\r
+>=20=20\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 +182,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 =3D 1;\r
+>      int i;\r
+>=20=20\r
+>      if (offset < 0) {\r
+> @@ -310,7 +194,7 @@ do_search_messages (const search_format_t *format,\r
+>      if (messages =3D=3D NULL)\r
+>      return 1;\r
+>=20=20\r
+> -    fputs (format->results_start, stdout);\r
+> +    format->begin_list (format);\r
+>=20=20\r
+>      for (i =3D 0;\r
+>       notmuch_messages_valid (messages) && (limit < 0 || i < offset + limit);\r
+> @@ -328,24 +212,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 =3D 0;\r
+> +            format->string (format, notmuch_filenames_get (filenames));\r
+> +            format->separator (format);\r
+>          }\r
+>=20=20=09=20=20=20=20\r
+>          notmuch_filenames_destroy( filenames );\r
+>=20=20\r
+>      } else { /* output =3D=3D 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 =3D 0;\r
+> +        format->set_prefix (format, "id");\r
+> +        format->string (format,\r
+> +                        notmuch_message_get_message_id (message));\r
+> +        format->separator (format);\r
+>      }\r
+>=20=20\r
+>      notmuch_message_destroy (message);\r
+> @@ -353,23 +230,19 @@ do_search_messages (const search_format_t *format,\r
+>=20=20\r
+>      notmuch_messages_destroy (messages);\r
+>=20=20\r
+> -    if (first_message)\r
+> -    fputs (format->results_null, stdout);\r
+> -    else\r
+> -    fputs (format->results_end, stdout);\r
+> +    format->end (format);\r
+>=20=20\r
+>      return 0;\r
+>  }\r
+>=20=20\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 =3D NULL;\r
+>      notmuch_tags_t *tags;\r
+>      const char *tag;\r
+> -    int first_tag =3D 1;\r
+>=20=20\r
+>      /* should the following only special case if no excluded terms\r
+>       * specified? */\r
+> @@ -387,7 +260,7 @@ do_search_tags (notmuch_database_t *notmuch,\r
+>      if (tags =3D=3D NULL)\r
+>      return 1;\r
+>=20=20\r
+> -    fputs (format->results_start, stdout);\r
+> +    format->begin_list (format);\r
+>=20=20\r
+>      for (;\r
+>       notmuch_tags_valid (tags);\r
+> @@ -395,12 +268,9 @@ do_search_tags (notmuch_database_t *notmuch,\r
+>      {\r
+>      tag =3D notmuch_tags_get (tags);\r
+>=20=20\r
+> -    if (! first_tag)\r
+> -        fputs (format->item_sep, stdout);\r
+> +    format->string (format, tag);\r
+> +    format->separator (format);\r
+>=20=20\r
+> -    format->item_id (tags, "", tag);\r
+> -\r
+> -    first_tag =3D 0;\r
+>      }\r
+>=20=20\r
+>      notmuch_tags_destroy (tags);\r
+> @@ -408,10 +278,7 @@ do_search_tags (notmuch_database_t *notmuch,\r
+>      if (messages)\r
+>      notmuch_messages_destroy (messages);\r
+>=20=20\r
+> -    if (first_tag)\r
+> -    fputs (format->results_null, stdout);\r
+> -    else\r
+> -    fputs (format->results_end, stdout);\r
+> +    format->end (format);\r
+>=20=20\r
+>      return 0;\r
+>  }\r
+> @@ -430,7 +297,7 @@ notmuch_search_command (void *ctx, int argc, char *ar=\r
+gv[])\r
+>      notmuch_query_t *query;\r
+>      char *query_str;\r
+>      notmuch_sort_t sort =3D NOTMUCH_SORT_NEWEST_FIRST;\r
+> -    const search_format_t *format =3D &format_text;\r
+> +    sprinter_t *format =3D NULL;\r
+\r
+due to this change -- dropping "fallback" setting due to ...\r
+\r
+>      int opt_index, ret;\r
+>      output_t output =3D OUTPUT_SUMMARY;\r
+>      int offset =3D 0;\r
+> @@ -475,10 +342,10 @@ notmuch_search_command (void *ctx, int argc, char *=\r
+argv[])\r
+>=20=20\r
+>      switch (format_sel) {\r
+>      case NOTMUCH_FORMAT_TEXT:\r
+> -    format =3D &format_text;\r
+> +    format =3D sprinter_text_create (ctx, stdout);\r
+>      break;\r
+>      case NOTMUCH_FORMAT_JSON:\r
+> -    format =3D &format_json;\r
+> +    format =3D sprinter_json_create (ctx, stdout);\r
+>      break;\r
+>      }\r
+\r
+... the dynamically created structure above, should we have 'default:'\r
+case which executes INTERNAL_ERROR() (or something) instead of allowing\r
+NULL derefencing (and crash) if one ever encounters such a situation ?\r
+\r
+Tomi\r
+\r
+\r
+> @@ -546,5 +413,7 @@ notmuch_search_command (void *ctx, int argc, char *ar=\r
+gv[])\r
+>      notmuch_query_destroy (query);\r
+>      notmuch_database_destroy (notmuch);\r
+>=20=20\r
+> +    talloc_free (format);\r
+> +\r
+>      return ret;\r
+>  }\r
+> diff --git a/test/json b/test/json\r
+> index 6439788..337b3f5 100755\r
+> --- a/test/json\r
+> +++ b/test/json\r
+> @@ -9,15 +9,16 @@ test_expect_equal "$output" "[[[{\"id\": \"${gen_msg_id=\r
+}\", \"match\": true, \"e\r
+>=20=20\r
+>  test_begin_subtest "Search message: json"\r
+>  add_message "[subject]=3D\"json-search-subject\"" "[date]=3D\"Sat, 01 Ja=\r
+n 2000 12:00:00 -0000\"" "[body]=3D\"json-search-message\""\r
+> -output=3D$(notmuch search --format=3Djson "json-search-message" | notmuc=\r
+h_search_sanitize)\r
+> +output=3D$(notmuch search --format=3Djson "json-search-message" | notmuc=\r
+h_json_show_sanitize | 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-subject\",\r
+> -\"tags\": [\"inbox\", \"unread\"]}]"\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\",\r
+> + \"unread\"]}]"\r
+>=20=20\r
+>  test_begin_subtest "Show message: json, utf-8"\r
+>  add_message "[subject]=3D\"json-show-utf8-body-s=C3=BCbj=C3=A9ct\"" "[da=\r
+te]=3D\"Sat, 01 Jan 2000 12:00:00 -0000\"" "[body]=3D\"js=C3=B6n-show-m=C3=\r
+=A9ssage\""\r
+> @@ -39,14 +40,15 @@ test_expect_equal "$output" "[[[{\"id\": \"$id\", \"m=\r
+atch\": true, \"excluded\":\r
+>=20=20\r
+>  test_begin_subtest "Search message: json, utf-8"\r
+>  add_message "[subject]=3D\"json-search-utf8-body-s=C3=BCbj=C3=A9ct\"" "[=\r
+date]=3D\"Sat, 01 Jan 2000 12:00:00 -0000\"" "[body]=3D\"js=C3=B6n-search-m=\r
+=C3=A9ssage\""\r
+> -output=3D$(notmuch search --format=3Djson "js=C3=B6n-search-m=C3=A9ssage=\r
+" | notmuch_search_sanitize)\r
+> +output=3D$(notmuch search --format=3Djson "js=C3=B6n-search-m=C3=A9ssage=\r
+" | notmuch_json_show_sanitize | 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=C3=BCbj=C3=A9ct\",\r
+> -\"tags\": [\"inbox\", \"unread\"]}]"\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=C3=BCbj=C3=A9ct\",\r
+> + \"tags\": [\"inbox\",\r
+> + \"unread\"]}]"\r
+>=20=20\r
+>  test_done\r
+> --=20\r
+> 1.7.11.2\r
+>\r
+> _______________________________________________\r
+> notmuch mailing list\r
+> notmuch@notmuchmail.org\r
+> http://notmuchmail.org/mailman/listinfo/notmuch\r