Re: [PATCH v2 2/4] reply: Add a JSON reply format.
authorJani Nikula <jani@nikula.org>
Wed, 18 Jan 2012 23:07:02 +0000 (01:07 +0200)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:42:39 +0000 (09:42 -0800)
6b/3faf3d17ecaae8594cd335afe4d4d899729f38 [new file with mode: 0644]

diff --git a/6b/3faf3d17ecaae8594cd335afe4d4d899729f38 b/6b/3faf3d17ecaae8594cd335afe4d4d899729f38
new file mode 100644 (file)
index 0000000..e54cf11
--- /dev/null
@@ -0,0 +1,528 @@
+Return-Path: <jani@nikula.org>\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 B19CA431FAF\r
+       for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 15:07:09 -0800 (PST)\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 LNvcW-t9nPoF for <notmuch@notmuchmail.org>;\r
+       Wed, 18 Jan 2012 15:07:08 -0800 (PST)\r
+Received: from mail-ey0-f181.google.com (mail-ey0-f181.google.com\r
+       [209.85.215.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id B5EC5431FAE\r
+       for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 15:07:07 -0800 (PST)\r
+Received: by eaal1 with SMTP id l1so499610eaa.26\r
+       for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 15:07:06 -0800 (PST)\r
+Received: by 10.213.16.132 with SMTP id o4mr1519710eba.88.1326928026594;\r
+       Wed, 18 Jan 2012 15:07:06 -0800 (PST)\r
+Received: from localhost (dsl-hkibrasgw4-fe50f800-253.dhcp.inet.fi.\r
+       [84.248.80.253])\r
+       by mx.google.com with ESMTPS id x4sm10718473eeb.4.2012.01.18.15.07.04\r
+       (version=SSLv3 cipher=OTHER); Wed, 18 Jan 2012 15:07:05 -0800 (PST)\r
+From: Jani Nikula <jani@nikula.org>\r
+To: Adam Wolfe Gordon <awg+notmuch@xvx.ca>, notmuch@notmuchmail.org\r
+Subject: Re: [PATCH v2 2/4] reply: Add a JSON reply format.\r
+In-Reply-To: <1326737603-21166-3-git-send-email-awg+notmuch@xvx.ca>\r
+References: <1326737603-21166-1-git-send-email-awg+notmuch@xvx.ca>\r
+       <1326737603-21166-3-git-send-email-awg+notmuch@xvx.ca>\r
+User-Agent: Notmuch/0.11+76~g1de742d (http://notmuchmail.org) Emacs/23.3.1\r
+       (i686-pc-linux-gnu)\r
+Date: Thu, 19 Jan 2012 01:07:02 +0200\r
+Message-ID: <87sjjcbofd.fsf@nikula.org>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=us-ascii\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 Jan 2012 23:07:09 -0000\r
+\r
+On Mon, 16 Jan 2012 11:13:21 -0700, Adam Wolfe Gordon <awg+notmuch@xvx.ca> wrote:\r
+> This new JSON format for replies includes headers generated for a reply\r
+> message as well as the headers and all text parts of the original message.\r
+> Using this data, a client can intelligently create a reply. For example,\r
+> the emacs client will be able to create replies with quoted HTML parts by\r
+> parsing the HTML parts using w3m.\r
+\r
+Hi, admittedly not a very thorough review, but please find a couple of\r
+comments below.\r
+\r
+BR,\r
+Jani.\r
+\r
+\r
+> ---\r
+>  notmuch-reply.c |  313 ++++++++++++++++++++++++++++++++++++++++++++-----------\r
+>  1 files changed, 253 insertions(+), 60 deletions(-)\r
+> \r
+> diff --git a/notmuch-reply.c b/notmuch-reply.c\r
+> index da3acce..f5a5dcf 100644\r
+> --- a/notmuch-reply.c\r
+> +++ b/notmuch-reply.c\r
+> @@ -30,6 +30,15 @@ reply_headers_message_part (GMimeMessage *message);\r
+>  static void\r
+>  reply_part_content (GMimeObject *part);\r
+>  \r
+> +static void\r
+> +reply_part_start_json (GMimeObject *part, int *part_count);\r
+> +\r
+> +static void\r
+> +reply_part_content_json (GMimeObject *part);\r
+> +\r
+> +static void\r
+> +reply_part_end_json (GMimeObject *part);\r
+> +\r
+>  static const notmuch_show_format_t format_reply = {\r
+>      "",\r
+>      "", NULL,\r
+> @@ -46,6 +55,22 @@ static const notmuch_show_format_t format_reply = {\r
+>      ""\r
+>  };\r
+>  \r
+> +static const notmuch_show_format_t format_json = {\r
+> +    "",\r
+> +    "", NULL,\r
+> +        "", NULL, NULL, "",\r
+> +        "",\r
+> +            reply_part_start_json,\r
+> +            NULL,\r
+> +            NULL,\r
+> +            reply_part_content_json,\r
+> +            reply_part_end_json,\r
+> +            "",\r
+> +        "",\r
+> +    "", "",\r
+> +    ""\r
+> +};\r
+> +\r
+>  static void\r
+>  show_reply_headers (GMimeMessage *message)\r
+>  {\r
+> @@ -54,14 +79,14 @@ show_reply_headers (GMimeMessage *message)\r
+>      stream_stdout = g_mime_stream_file_new (stdout);\r
+>      if (stream_stdout) {\r
+>      g_mime_stream_file_set_owner (GMIME_STREAM_FILE (stream_stdout), FALSE);\r
+> -    stream_filter = g_mime_stream_filter_new(stream_stdout);\r
+> +    stream_filter = g_mime_stream_filter_new (stream_stdout);\r
+>      if (stream_filter) {\r
+> -            g_mime_stream_filter_add(GMIME_STREAM_FILTER(stream_filter),\r
+> -                                     g_mime_filter_headers_new());\r
+> -            g_mime_object_write_to_stream(GMIME_OBJECT(message), stream_filter);\r
+> -            g_object_unref(stream_filter);\r
+> +            g_mime_stream_filter_add (GMIME_STREAM_FILTER(stream_filter),\r
+> +                                      g_mime_filter_headers_new());\r
+> +            g_mime_object_write_to_stream (GMIME_OBJECT(message), stream_filter);\r
+> +            g_object_unref (stream_filter);\r
+>      }\r
+> -    g_object_unref(stream_stdout);\r
+> +    g_object_unref (stream_stdout);\r
+\r
+I know I asked you to adhere to notmuch coding style like above, but I\r
+meant in the context of your patch, not elsewhere. Cleanups like this\r
+should really be separate patches. Sorry if I was ambiguous.\r
+\r
+>      }\r
+>  }\r
+>  \r
+> @@ -86,6 +111,17 @@ reply_headers_message_part (GMimeMessage *message)\r
+>      printf ("> Date: %s\n", g_mime_message_get_date_as_string (message));\r
+>  }\r
+>  \r
+> +static notmuch_bool_t\r
+> +reply_check_part_type (GMimeObject *part, const char *type, const char *subtype,\r
+> +                   const char *disposition)\r
+> +{\r
+> +    GMimeContentType *content_type = g_mime_object_get_content_type (GMIME_OBJECT (part));\r
+> +    GMimeContentDisposition *part_disposition = g_mime_object_get_content_disposition (part);\r
+> +\r
+> +    return (g_mime_content_type_is_type (content_type, type, subtype) &&\r
+> +        (!part_disposition ||\r
+> +         strcmp (part_disposition->disposition, disposition) == 0));\r
+> +}\r
+>  \r
+>  static void\r
+>  reply_part_content (GMimeObject *part)\r
+> @@ -108,32 +144,29 @@ reply_part_content (GMimeObject *part)\r
+>      {\r
+>      GMimeStream *stream_stdout = NULL, *stream_filter = NULL;\r
+>      GMimeDataWrapper *wrapper;\r
+> -    const char *charset;\r
+> -\r
+> -    charset = g_mime_object_get_content_type_parameter (part, "charset");\r
+>      stream_stdout = g_mime_stream_file_new (stdout);\r
+>      if (stream_stdout) {\r
+>          g_mime_stream_file_set_owner (GMIME_STREAM_FILE (stream_stdout), FALSE);\r
+> -        stream_filter = g_mime_stream_filter_new(stream_stdout);\r
+> -        if (charset) {\r
+> -            g_mime_stream_filter_add(GMIME_STREAM_FILTER(stream_filter),\r
+> -                                     g_mime_filter_charset_new(charset, "UTF-8"));\r
+> -        }\r
+> +        stream_filter = g_mime_stream_filter_new (stream_stdout);\r
+> +\r
+> +        const char *charset = g_mime_object_get_content_type_parameter (part, "charset");\r
+> +        if (charset)\r
+> +            g_mime_stream_filter_add(GMIME_STREAM_FILTER (stream_filter),\r
+> +                                     g_mime_filter_charset_new (charset, "UTF-8"));\r
+>      }\r
+> -    g_mime_stream_filter_add(GMIME_STREAM_FILTER(stream_filter),\r
+> -                             g_mime_filter_reply_new(TRUE));\r
+> +    g_mime_stream_filter_add (GMIME_STREAM_FILTER (stream_filter),\r
+> +                              g_mime_filter_reply_new (TRUE));\r
+>      wrapper = g_mime_part_get_content_object (GMIME_PART (part));\r
+>      if (wrapper && stream_filter)\r
+>          g_mime_data_wrapper_write_to_stream (wrapper, stream_filter);\r
+>      if (stream_filter)\r
+> -        g_object_unref(stream_filter);\r
+> +        g_object_unref (stream_filter);\r
+>      if (stream_stdout)\r
+> -        g_object_unref(stream_stdout);\r
+> +        g_object_unref (stream_stdout);\r
+>      }\r
+>      else\r
+>      {\r
+> -    if (disposition &&\r
+> -        strcmp (disposition->disposition, GMIME_DISPOSITION_ATTACHMENT) == 0)\r
+> +    if (disposition && strcmp (disposition->disposition, GMIME_DISPOSITION_ATTACHMENT) == 0)\r
+\r
+This is also a change not related to your patch.\r
+\r
+>      {\r
+>          const char *filename = g_mime_part_get_filename (GMIME_PART (part));\r
+>          printf ("Attachment: %s (%s)\n", filename,\r
+> @@ -147,6 +180,67 @@ reply_part_content (GMimeObject *part)\r
+>      }\r
+>  }\r
+>  \r
+> +static void\r
+> +reply_part_start_json (GMimeObject *part, unused (int *part_count))\r
+> +{\r
+> +    if (reply_check_part_type (part, "text", "*", GMIME_DISPOSITION_INLINE))\r
+> +    printf ("{ ");\r
+> +}\r
+> +\r
+> +static void\r
+> +reply_part_end_json (GMimeObject *part)\r
+> +{\r
+> +    if (reply_check_part_type (part, "text", "*", GMIME_DISPOSITION_INLINE))\r
+> +    printf ("}, ");\r
+> +}\r
+> +\r
+> +static void\r
+> +reply_part_content_json (GMimeObject *part)\r
+> +{\r
+> +    GMimeContentType *content_type = g_mime_object_get_content_type (GMIME_OBJECT (part));\r
+> +    void *ctx = talloc_new (NULL);\r
+> +\r
+> +    /* We only care about inline text parts for reply purposes */\r
+> +    if (reply_check_part_type (part, "text", "*", GMIME_DISPOSITION_INLINE))\r
+> +    {\r
+\r
+The style in notmuch is to put the opening brace in the end of the if\r
+line. Same applies below.\r
+\r
+> +    GMimeDataWrapper *wrapper;\r
+> +    GByteArray *part_content;\r
+> +\r
+> +    printf ("\"content-type\": %s, \"content\": ",\r
+> +           json_quote_str (ctx, g_mime_content_type_to_string (content_type)));\r
+> +\r
+> +    wrapper = g_mime_part_get_content_object (GMIME_PART (part));\r
+> +    if (wrapper)\r
+> +    {\r
+> +        const char *charset = g_mime_object_get_content_type_parameter (part, "charset");\r
+> +        GMimeStream *stream_memory = g_mime_stream_mem_new ();\r
+> +        if (stream_memory) {\r
+> +            GMimeStream *stream_filter = NULL;\r
+> +            stream_filter = g_mime_stream_filter_new (stream_memory);\r
+> +            if (charset) {\r
+> +                g_mime_stream_filter_add (GMIME_STREAM_FILTER (stream_filter),\r
+> +                                          g_mime_filter_charset_new (charset, "UTF-8"));\r
+> +            }\r
+> +\r
+> +            if (stream_filter)\r
+> +            {\r
+> +                g_mime_data_wrapper_write_to_stream (wrapper, stream_filter);\r
+> +                part_content = g_mime_stream_mem_get_byte_array (GMIME_STREAM_MEM (stream_memory));\r
+> +\r
+> +                printf ("%s", json_quote_chararray (ctx, (char *) part_content->data, part_content->len));\r
+> +            }\r
+> +            if (stream_filter)\r
+> +                g_object_unref (stream_filter);\r
+> +        }\r
+> +\r
+> +        if (stream_memory)\r
+> +            g_object_unref (stream_memory);\r
+> +    }\r
+> +    }\r
+> +\r
+> +    talloc_free (ctx);\r
+> +}\r
+> +\r
+>  /* Is the given address configured as one of the user's "personal" or\r
+>   * "other" addresses. */\r
+>  static int\r
+> @@ -505,6 +599,61 @@ guess_from_received_header (notmuch_config_t *config, notmuch_message_t *message\r
+>      return NULL;\r
+>  }\r
+>  \r
+> +static GMimeMessage *\r
+> +create_reply_message(void *ctx,\r
+> +                 notmuch_config_t *config,\r
+> +                 notmuch_message_t *message,\r
+> +                 notmuch_bool_t reply_all)\r
+> +{\r
+> +    const char *subject, *from_addr = NULL;\r
+> +    const char *in_reply_to, *orig_references, *references;\r
+> +\r
+> +    /* The 1 means we want headers in a "pretty" order. */\r
+> +    GMimeMessage *reply = g_mime_message_new (1);\r
+> +    if (reply == NULL) {\r
+> +    fprintf (stderr, "Out of memory\n");\r
+> +    return NULL;\r
+> +    }\r
+> +\r
+> +    subject = notmuch_message_get_header (message, "subject");\r
+> +    if (subject) {\r
+> +    if (strncasecmp (subject, "Re:", 3))\r
+> +        subject = talloc_asprintf (ctx, "Re: %s", subject);\r
+> +    g_mime_message_set_subject (reply, subject);\r
+> +    }\r
+> +\r
+> +    from_addr = add_recipients_from_message (reply, config,\r
+> +                                         message, reply_all);\r
+> +\r
+> +    if (from_addr == NULL)\r
+> +    from_addr = guess_from_received_header (config, message);\r
+> +\r
+> +    if (from_addr == NULL)\r
+> +    from_addr = notmuch_config_get_user_primary_email (config);\r
+> +\r
+> +    from_addr = talloc_asprintf (ctx, "%s <%s>",\r
+> +                             notmuch_config_get_user_name (config),\r
+> +                             from_addr);\r
+> +    g_mime_object_set_header (GMIME_OBJECT (reply),\r
+> +                          "From", from_addr);\r
+> +\r
+> +    in_reply_to = talloc_asprintf (ctx, "<%s>",\r
+> +                               notmuch_message_get_message_id (message));\r
+> +\r
+> +    g_mime_object_set_header (GMIME_OBJECT (reply),\r
+> +                          "In-Reply-To", in_reply_to);\r
+> +\r
+> +    orig_references = notmuch_message_get_header (message, "references");\r
+> +    references = talloc_asprintf (ctx, "%s%s%s",\r
+> +                              orig_references ? orig_references : "",\r
+> +                              orig_references ? " " : "",\r
+> +                              in_reply_to);\r
+> +    g_mime_object_set_header (GMIME_OBJECT (reply),\r
+> +                          "References", references);\r
+> +\r
+> +    return reply;\r
+> +}\r
+> +\r
+>  static int\r
+>  notmuch_reply_format_default(void *ctx,\r
+>                           notmuch_config_t *config,\r
+> @@ -515,8 +664,6 @@ notmuch_reply_format_default(void *ctx,\r
+>      GMimeMessage *reply;\r
+>      notmuch_messages_t *messages;\r
+>      notmuch_message_t *message;\r
+> -    const char *subject, *from_addr = NULL;\r
+> -    const char *in_reply_to, *orig_references, *references;\r
+>      const notmuch_show_format_t *format = &format_reply;\r
+>  \r
+>      for (messages = notmuch_query_search_messages (query);\r
+> @@ -525,62 +672,104 @@ notmuch_reply_format_default(void *ctx,\r
+>      {\r
+>      message = notmuch_messages_get (messages);\r
+>  \r
+> -    /* The 1 means we want headers in a "pretty" order. */\r
+> -    reply = g_mime_message_new (1);\r
+> -    if (reply == NULL) {\r
+> -        fprintf (stderr, "Out of memory\n");\r
+> -        return 1;\r
+> -    }\r
+> +    reply = create_reply_message (ctx, config, message, reply_all);\r
+>  \r
+> -    subject = notmuch_message_get_header (message, "subject");\r
+> -    if (subject) {\r
+> -        if (strncasecmp (subject, "Re:", 3))\r
+> -            subject = talloc_asprintf (ctx, "Re: %s", subject);\r
+> -        g_mime_message_set_subject (reply, subject);\r
+> -    }\r
+> +    if (!reply)\r
+> +        continue;\r
+>  \r
+> -    from_addr = add_recipients_from_message (reply, config, message,\r
+> -                                             reply_all);\r
+> +    show_reply_headers (reply);\r
+>  \r
+> -    if (from_addr == NULL)\r
+> -        from_addr = guess_from_received_header (config, message);\r
+> +    g_object_unref (G_OBJECT (reply));\r
+> +    reply = NULL;\r
+>  \r
+> -    if (from_addr == NULL)\r
+> -        from_addr = notmuch_config_get_user_primary_email (config);\r
+> +    printf ("On %s, %s wrote:\n",\r
+> +            notmuch_message_get_header (message, "date"),\r
+> +            notmuch_message_get_header (message, "from"));\r
+>  \r
+> -    from_addr = talloc_asprintf (ctx, "%s <%s>",\r
+> -                                 notmuch_config_get_user_name (config),\r
+> -                                 from_addr);\r
+> -    g_mime_object_set_header (GMIME_OBJECT (reply),\r
+> -                              "From", from_addr);\r
+> +    show_message_body (message, format, params);\r
+>  \r
+> -    in_reply_to = talloc_asprintf (ctx, "<%s>",\r
+> -                         notmuch_message_get_message_id (message));\r
+> +    notmuch_message_destroy (message);\r
+> +    }\r
+> +    return 0;\r
+> +}\r
+>  \r
+> -    g_mime_object_set_header (GMIME_OBJECT (reply),\r
+> -                              "In-Reply-To", in_reply_to);\r
+> +static int\r
+> +notmuch_reply_format_json(void *ctx,\r
+> +                      notmuch_config_t *config,\r
+> +                      notmuch_query_t *query,\r
+> +                      unused (notmuch_show_params_t *params),\r
+> +                      notmuch_bool_t reply_all)\r
+> +{\r
+> +    GMimeMessage *reply;\r
+> +    notmuch_messages_t *messages;\r
+> +    notmuch_message_t *message;\r
+> +    const notmuch_show_format_t *format = &format_json;\r
+>  \r
+> -    orig_references = notmuch_message_get_header (message, "references");\r
+> -    references = talloc_asprintf (ctx, "%s%s%s",\r
+> -                                  orig_references ? orig_references : "",\r
+> -                                  orig_references ? " " : "",\r
+> -                                  in_reply_to);\r
+> -    g_mime_object_set_header (GMIME_OBJECT (reply),\r
+> -                              "References", references);\r
+> +    const char *reply_headers[] = {"from", "to", "subject", "in-reply-to", "references"};\r
+> +    const char *orig_headers[] = {"from", "to", "cc", "subject", "date", "in-reply-to", "references"};\r
+> +    unsigned int hidx;\r
+>  \r
+> -    show_reply_headers (reply);\r
+> +    /* Start array of reply objects */\r
+> +    printf ("[");\r
+> +\r
+> +    for (messages = notmuch_query_search_messages (query);\r
+> +     notmuch_messages_valid (messages);\r
+> +     notmuch_messages_move_to_next (messages))\r
+> +    {\r
+> +    /* Start a reply object */\r
+> +    printf ("{ \"reply\": { \"headers\": { ");\r
+> +\r
+> +    message = notmuch_messages_get (messages);\r
+> +\r
+> +    reply = create_reply_message (ctx, config, message, reply_all);\r
+> +    if (!reply)\r
+> +        continue;\r
+\r
+If this occurs, it will create broken JSON.\r
+\r
+> +\r
+> +    for (hidx = 0; hidx < ARRAY_SIZE (reply_headers); hidx++)\r
+> +    {\r
+> +        if (hidx)\r
+> +            printf (", ");\r
+> +\r
+> +        printf ("%s: %s", json_quote_str (ctx, reply_headers[hidx]),\r
+> +                json_quote_str (ctx, g_mime_object_get_header (GMIME_OBJECT (reply), reply_headers[hidx])));\r
+> +    }\r
+>  \r
+>      g_object_unref (G_OBJECT (reply));\r
+>      reply = NULL;\r
+>  \r
+> -    printf ("On %s, %s wrote:\n",\r
+> -            notmuch_message_get_header (message, "date"),\r
+> -            notmuch_message_get_header (message, "from"));\r
+> +    /* Done the headers for the reply, which has no body parts */\r
+> +    printf ("} }");\r
+> +\r
+> +    /* Start the original */\r
+> +    printf (", \"original\": { \"headers\": { ");\r
+> +\r
+> +    for (hidx = 0; hidx < ARRAY_SIZE (orig_headers); hidx++)\r
+> +    {\r
+> +        if (hidx)\r
+> +            printf (", ");\r
+> +\r
+> +        printf ("%s: %s", json_quote_str (ctx, orig_headers[hidx]),\r
+> +                json_quote_str (ctx, notmuch_message_get_header (message, orig_headers[hidx])));\r
+> +    }\r
+> +\r
+> +    /* End headers */\r
+> +    printf (" }, \"body\": [ ");\r
+>  \r
+> +    /* Show body parts */\r
+>      show_message_body (message, format, params);\r
+>  \r
+>      notmuch_message_destroy (message);\r
+> +\r
+> +    /* Done the original */\r
+> +    printf ("{} ] }");\r
+> +\r
+> +    /* End the reply object. */\r
+> +    printf (" }, ");\r
+>      }\r
+> +\r
+> +    /* End array of reply objects */\r
+> +    printf ("{} ]\n");\r
+> +\r
+>      return 0;\r
+>  }\r
+>  \r
+> @@ -646,6 +835,7 @@ notmuch_reply_format_headers_only(void *ctx,\r
+>  \r
+>  enum {\r
+>      FORMAT_DEFAULT,\r
+> +    FORMAT_JSON,\r
+>      FORMAT_HEADERS_ONLY,\r
+>  };\r
+>  \r
+> @@ -666,6 +856,7 @@ notmuch_reply_command (void *ctx, int argc, char *argv[])\r
+>      notmuch_opt_desc_t options[] = {\r
+>      { NOTMUCH_OPT_KEYWORD, &format, "format", 'f',\r
+>        (notmuch_keyword_t []){ { "default", FORMAT_DEFAULT },\r
+> +                              { "json", FORMAT_JSON },\r
+>                                { "headers-only", FORMAT_HEADERS_ONLY },\r
+>                                { 0, 0 } } },\r
+>      { NOTMUCH_OPT_KEYWORD, &reply_all, "reply-to", 'r',\r
+> @@ -684,6 +875,8 @@ notmuch_reply_command (void *ctx, int argc, char *argv[])\r
+>  \r
+>      if (format == FORMAT_HEADERS_ONLY)\r
+>      reply_format_func = notmuch_reply_format_headers_only;\r
+> +    else if (format == FORMAT_JSON)\r
+> +    reply_format_func = notmuch_reply_format_json;\r
+>      else\r
+>      reply_format_func = notmuch_reply_format_default;\r
+>  \r
+> -- \r
+> 1.7.5.4\r
+> \r
+> _______________________________________________\r
+> notmuch mailing list\r
+> notmuch@notmuchmail.org\r
+> http://notmuchmail.org/mailman/listinfo/notmuch\r