b9/a254a5f88aa60fd79b3d00dc4535bf2df71d71

   1 Return-Path: <jani@nikula.org>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 7C170431FB6\r
   6         for <notmuch@notmuchmail.org>; Tue,  4 Dec 2012 09:07:58 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id Y43z74OHqf+N for <notmuch@notmuchmail.org>;\r
  16         Tue,  4 Dec 2012 09:07:54 -0800 (PST)\r
  17 Received: from mail-la0-f53.google.com (mail-la0-f53.google.com\r
  18         [209.85.215.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 3C076431FAE\r
  21         for <notmuch@notmuchmail.org>; Tue,  4 Dec 2012 09:07:54 -0800 (PST)\r
  22 Received: by mail-la0-f53.google.com with SMTP id w12so3572120lag.26\r
  23         for <notmuch@notmuchmail.org>; Tue, 04 Dec 2012 09:07:52 -0800 (PST)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  27         :mime-version:content-type:x-gm-message-state;\r
  28         bh=44fUyli5oPeB2mTyDhZCSircK2AGRbDBM0Z85iCRyaI=;\r
  29         b=elREwdkk31y5oqSClhUI1uKRvn86Iu2dkAlyBfhSk1+hDuIM+bu+zZVIhkG4kg8I7A\r
  30         MYlmZ9vUxqrhynM92Uu39uVmqyijoKoWx9+NP4XOYB/nG721WsOpTL0qUipXGapj3PFc\r
  31         2Pv61aLW+RK0FTi4AKCYsqwwkAwaQrfDkZI+892x2pa+PecfDHGjQDPz6Y3zxYga3nzL\r
  32         3GJf8DksvU3HZwBh03tQxd85Xue8vDl8pLQNLLkr5Btn84GjuxDso5EkM+oGm0pOzNGo\r
  33         N+dNaOOwesFcGUXZxqUdhrhrgoozEAvXKzr2gffHj5QKzc18frDUj58L/Rhlqx2CVdVq\r
  34         hOfQ==\r
  35 Received: by 10.152.144.10 with SMTP id si10mr13623174lab.55.1354640872324;\r
  36         Tue, 04 Dec 2012 09:07:52 -0800 (PST)\r
  37 Received: from localhost (dsl-hkibrasgw4-fe51df00-27.dhcp.inet.fi.\r
  38         [80.223.81.27])\r
  39         by mx.google.com with ESMTPS id pw17sm976447lab.5.2012.12.04.09.07.47\r
  40         (version=SSLv3 cipher=OTHER); Tue, 04 Dec 2012 09:07:49 -0800 (PST)\r
  41 From: Jani Nikula <jani@nikula.org>\r
  42 To: Peter Feigl <craven@gmx.net>, notmuch@notmuchmail.org\r
  43 Subject: Re: [PATCH v2 0/5] New output format sexp (Lisp S-Expressions)\r
  44 In-Reply-To: <1354632382-15609-1-git-send-email-craven@gmx.net>\r
  45 References: <1354264143-30173-1-git-send-email-craven@gmx.net>\r
  46         <1354632382-15609-1-git-send-email-craven@gmx.net>\r
  47 User-Agent: Notmuch/0.14+137~g9203c35 (http://notmuchmail.org) Emacs/23.4.1\r
  48         (i686-pc-linux-gnu)\r
  49 Date: Tue, 04 Dec 2012 19:07:46 +0200\r
  50 Message-ID: <87boe9lp99.fsf@nikula.org>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: text/plain; charset=us-ascii\r
  53 X-Gm-Message-State:\r
  54  ALoCoQk0ZWefhJrj2fR85I7MfObeMixCWihiyysk8Ete6TWXD7wcrszqd5xX2NCrFXSNgC+3egrg\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Tue, 04 Dec 2012 17:07:58 -0000\r
  68 \r
  69 \r
  70 Hi Peter -\r
  71 \r
  72 On Tue, 04 Dec 2012, Peter Feigl <craven@gmx.net> wrote:\r
  73 > This patch series adds a new output format "sexp" to notmuch-reply,\r
  74 > notmuch-show and notmuch-search. These are useful for the Android mobile\r
  75 > client and perhaps other Lisp programs as well.\r
  76 > After the switch to a generic structured output printer, which was\r
  77 > committed some months ago, these patches just add another one (like the\r
  78 > json structured output printer).\r
  79 > Basic tests and updates to the man pages are also included.\r
  80 >\r
  81 >\r
  82 > Peter Feigl (5):\r
  83 >   Adding an S-expression structured output printer.\r
  84 >   Rename the -json printer functions in notmuch-reply and    \r
  85 >     notmuch-show to generic -sprinter functions.\r
  86 >   Use the S-Expression structured printer in notmuch-show,\r
  87 >     notmuch-reply     and notmuch-search.\r
  88 \r
  89 Patches 1-3 look good.\r
  90 \r
  91 >   Adding tests for --format=sexp.\r
  92 \r
  93 Did not review.\r
  94 \r
  95 >   Updating man pages for new S-Expression output format.\r
  96 \r
  97 Did not review, but noticed it adds lines with trailing whitespace.\r
  98 \r
  99 \r
 100 BR,\r
 101 Jani.\r
 102 \r
 103 \r
 104 >\r
 105 >  Makefile.local            |   1 +\r
 106 >  man/man1/notmuch-reply.1  |  14 ++-\r
 107 >  man/man1/notmuch-search.1 |  15 +--\r
 108 >  man/man1/notmuch-show.1   |  36 +++++--\r
 109 >  notmuch-client.h          |   8 +-\r
 110 >  notmuch-reply.c           |  43 ++++----\r
 111 >  notmuch-search.c          |   6 +-\r
 112 >  notmuch-show.c            |  48 +++++----\r
 113 >  sprinter-sexp.c           | 250 ++++++++++++++++++++++++++++++++++++++++++++++\r
 114 >  sprinter.h                |   4 +\r
 115 >  test/notmuch-test         |   1 +\r
 116 >  test/sexp                 |  48 +++++++++\r
 117 >  12 files changed, 414 insertions(+), 60 deletions(-)\r
 118 >  create mode 100644 sprinter-sexp.c\r
 119 >  create mode 100755 test/sexp\r
 120 >\r
 121 > -- \r
 122 > 1.8.0\r
 123 >\r
 124 > _______________________________________________\r
 125 > notmuch mailing list\r
 126 > notmuch@notmuchmail.org\r
 127 > http://notmuchmail.org/mailman/listinfo/notmuch\r