b4/d9c3a7768ef08ea107408894e6c6a634287d56

   1 Return-Path: <craven@gmx.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 3B33E429E40\r
   6         for <notmuch@notmuchmail.org>; Sat, 21 Jan 2012 15:21:40 -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.001\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.001 tagged_above=-999 required=5\r
  12         tests=[FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001]\r
  13         autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id MzB9rJSuimGD for <notmuch@notmuchmail.org>;\r
  17         Sat, 21 Jan 2012 15:21:39 -0800 (PST)\r
  18 Received: from mailout-de.gmx.net (mailout-de.gmx.net [213.165.64.23])\r
  19         by olra.theworths.org (Postfix) with SMTP id 53454431FAF\r
  20         for <notmuch@notmuchmail.org>; Sat, 21 Jan 2012 15:21:39 -0800 (PST)\r
  21 Received: (qmail invoked by alias); 21 Jan 2012 23:21:38 -0000\r
  22 Received: from www.nexoid.at (EHLO mail.nexoid.at) [178.79.130.240]\r
  23         by mail.gmx.net (mp063) with SMTP; 22 Jan 2012 00:21:38 +0100\r
  24 X-Authenticated: #201305\r
  25 X-Provags-ID: V01U2FsdGVkX18HvhdUyeMHBoC6C3xaSRUeScItUw9hq8q0tWB09i\r
  26         EegPSslarHYe1z\r
  27 Received: by mail.nexoid.at (Postfix, from userid 1000)\r
  28         id 927FA57019; Sun, 22 Jan 2012 00:21:37 +0100 (CET)\r
  29 From: "Peter Feigl" <craven@gmx.net>\r
  30 To: Jameson Graef Rollins <jrollins@finestructure.net>\r
  31 Subject: Re: [PATCH] rewriting notmuch-search for structured output to make\r
  32         other output formats easier\r
  33 In-Reply-To: <87k44klkee.fsf@servo.finestructure.net>\r
  34 References: <1327180568-30385-1-git-send-email-craven@gmx.net>\r
  35         <87k44klkee.fsf@servo.finestructure.net>\r
  36 User-Agent: Notmuch/0.11+77~gad6d0d5 (http://notmuchmail.org) Emacs/23.3.1\r
  37         (i686-pc-linux-gnu)\r
  38 Date: Sun, 22 Jan 2012 00:21:37 +0100\r
  39 Message-ID: <87lip0ljzy.fsf@nexoid.at>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: text/plain; charset=us-ascii\r
  42 X-Y-GMX-Trusted: 0\r
  43 Cc: notmuch@notmuchmail.org\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: Sat, 21 Jan 2012 23:21:40 -0000\r
  57 \r
  58 On Sat, 21 Jan 2012 15:12:57 -0800, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
  59 > On Sat, 21 Jan 2012 22:16:08 +0100, Peter Feigl <craven@gmx.net> wrote:\r
  60 > > The output routines have been rewritten so that logical structure\r
  61 > > (objects with key/value pairs, arrays, strings and numbers) are\r
  62 > > written instead of ad-hoc printfs. This allows for easier adaptation\r
  63 > > of other output formats, as only the routines that start/end an object\r
  64 > > etc. have to be rewritten. The logic is the same for all formats.\r
  65 > > The default text output is handled differently, special cases are\r
  66 > > inserted at the proper places, as it differs too much from the\r
  67 > > structured output.\r
  68 > \r
  69 > Hi, Peter.  Thanks for the contribution.\r
  70 > \r
  71 > There are a lot of changes in this patch so I think you need to think\r
  72 > about how you can break this up into multiple smaller and more atomic\r
  73 > patches.  In particular, the addition of the sexp output format needs to\r
  74 > definitely be in a separate patch from the restructuring of the output\r
  75 > formatting.  You also don't mention anywhere in the commit log that\r
  76 > you've added this new output format.  You'll also need to include\r
  77 > documentation and test suite updates.\r
  78 \r
  79 I'm sorry I forgot to mention that, it was mainly meant as a way to show\r
  80 that this is easily possible (i.e. that the formatting is decoupled from\r
  81 the logic, so that additional and different formats can be added without\r
  82 influencing the printing logic). It'd be easy to split this up. What\r
  83 kind of documentation should I include? \r
  84 The test suite should work fine, *if* it compares EXPECTED and OUTPUT\r
  85 not character-by-character, but rather by pretty-printing both the\r
  86 expected and the actual outputs by some JSON pretty-printer (like python\r
  87 -mjson.tool). I can of course provide additional test-cases for\r
  88 --format=sexp.\r
  89 \r
  90 How should I proceed on this? Re-submit the patch with the sexp-support\r
  91 removed and only JSON updated?\r
  92 \r
  93 Thanks!\r
  94 \r
  95 Peter\r