b6/b6036d9627911278f5d98ab28f11784b22d098

   1 Return-Path: <jrollins@finestructure.net>\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 7BC55431E64\r
   6         for <notmuch@notmuchmail.org>; Tue, 10 Jul 2012 09:58:29 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 KUyiQggz+RSM for <notmuch@notmuchmail.org>;\r
  16         Tue, 10 Jul 2012 09:58:29 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 0C12D431FBF\r
  20         for <notmuch@notmuchmail.org>; Tue, 10 Jul 2012 09:58:29 -0700 (PDT)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id 88E40328094;\r
  23         Tue, 10 Jul 2012 09:58:28 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
  25 Received: from finestructure.net (adsl-76-240-184-246.dsl.irvnca.sbcglobal.net\r
  26         [76.240.184.246]) (Authenticated sender: jrollins)\r
  27         by fire-doxen-submit (Postfix) with ESMTP id 9A0AF2E5057E;\r
  28         Tue, 10 Jul 2012 09:58:25 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id BD5B2868; Tue, 10 Jul 2012 09:58:24 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: craven@gmx.net, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH] FIXED: Added better support for multiple structured\r
  34         output formats.\r
  35 In-Reply-To: <87fw8zvqmm.fsf@nexoid.at>\r
  36 References: <87liisue70.fsf@nexoid.at> <87fw8zvqmm.fsf@nexoid.at>\r
  37 User-Agent: Notmuch/0.13.2+54~ga0426dc (http://notmuchmail.org) Emacs/23.4.1\r
  38         (x86_64-pc-linux-gnu)\r
  39 Date: Tue, 10 Jul 2012 09:58:21 -0700\r
  40 Message-ID: <871ukjzh76.fsf@servo.finestructure.net>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: multipart/signed; boundary="=-=-=";\r
  43         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Tue, 10 Jul 2012 16:58:29 -0000\r
  57 \r
  58 --=-=-=\r
  59 \r
  60 On Tue, Jul 10 2012, craven@gmx.net wrote:\r
  61 > Sorry, the original patch I sent was missing a small part, here the full\r
  62 > patch:\r
  63 >\r
  64 > As discussed in <id:20120121220407.GK16740@mit.edu>, this patch adds\r
  65 > support for new structured output formats (like s-expressions) by using\r
  66 > stateful structure_printers. An implementation of the JSON structure\r
  67 > printer that passes all tests is included.\r
  68 >\r
  69 > Structure printers have functions for starting a map, a list, closing\r
  70 > any number of these two, printing a map key, a number value, a string\r
  71 > value, and boolean values. By threading a state through the\r
  72 > invocations, arbitrary structured formatting can be achieved.\r
  73 >\r
  74 > In a second patch, the structured output code should be isolated in a\r
  75 > separate file, and also used in all other parts of notmuch.\r
  76 \r
  77 Hi, Peter.  Thanks for submitting this.\r
  78 \r
  79 I would really like to see this patch broken up into multiple smaller\r
  80 and more atomic patches.  Smaller patches are much easier to read and\r
  81 review.  For instance, all the new formatting functions could be added\r
  82 in their own patch (and already in their own file if that's\r
  83 appropriate).  Then the existing json output could then be modified to\r
  84 use the new formatters.\r
  85 \r
  86 There is also json output in notmuch show and reply as well, so they\r
  87 should probably also be modified to use the new formatter.\r
  88 \r
  89 Thanks!\r
  90 \r
  91 jamie.\r
  92 \r
  93 --=-=-=\r
  94 Content-Type: application/pgp-signature\r
  95 \r
  96 -----BEGIN PGP SIGNATURE-----\r
  97 Version: GnuPG v1.4.12 (GNU/Linux)\r
  98 \r
  99 iQIcBAEBCAAGBQJP/F8tAAoJEO00zqvie6q8dFQP/2O4/dlWJsfLJfthyvy+D9eI\r
 100 RF7B9xELgo1gz/64BT33gd9+IVFWmdYc5MV/dmOYMlD8p96JFmEtS0bNTe1CDEL4\r
 101 hO3O48B2jSnQk/NBLNM90N9hAvu9Q/1Jnt5k93DD+PKZlmirkE988Id/FXT7NU+g\r
 102 zmyjeWqNCGP6mzDYcVng6J2oX+JhTbG/1yXhItwCqFxa+lwVShQ+9a5wL14winSY\r
 103 tAu8AXxlIN406OIIpMhiT2nSROHq9HZnCxJJvrsTS6ebrg+dqNqy0qzKpXic5vqo\r
 104 8NR8O/azNiJq3JaJeltMGQidyyt3y3lIT389uszj8Wv2XkwiKTLQToFRln5562Hx\r
 105 qRM4avz9oNEY+BdYNe1CsBKAA9O1kHsYRhJHO+3wk0nz3+ZYNDJnRT8pYLW4pFGx\r
 106 xghUYxsHVs+jUMze359diVA2r38voCdtOOSt/38fivaEq1M3MEMHsZeRAuul+ifu\r
 107 x+D0rB4u5iW/c2WituFuE5yNQZCUZYZGgQ1ppenq0BnD+kX3IeVjKwG1GJ404/VC\r
 108 LVEyZWg4yVQXhy7bL6T3JLcbej+oNQnGyg6CJZg6gdt8NR6HEfVTOb39oZmVab/S\r
 109 jm75eYbmq7MGEHChCK8fvJW3Cg3DzuszPTw9udpuSd19/JenaC8rGMTHk11IH+CF\r
 110 vH7AtjtimU+V44cWbLoB\r
 111 =wMZl\r
 112 -----END PGP SIGNATURE-----\r
 113 --=-=-=--\r