a8/092386b03fb1a1fea9887df9f8ce2e0580f633

   1 Return-Path: <tomi.ollila@iki.fi>\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 E0B1D431FBD\r
   6         for <notmuch@notmuchmail.org>; Thu, 23 Aug 2012 00:21:24 -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: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 qNMOzN+2L3Ih for <notmuch@notmuchmail.org>;\r
  16         Thu, 23 Aug 2012 00:21:24 -0700 (PDT)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 63BFA431FAE\r
  19         for <notmuch@notmuchmail.org>; Thu, 23 Aug 2012 00:21:24 -0700 (PDT)\r
  20 Received: by guru.guru-group.fi (Postfix, from userid 501)\r
  21         id 78BB0100386; Thu, 23 Aug 2012 10:21:33 +0300 (EEST)\r
  22 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  23 To: Jameson Graef Rollins <jrollins@finestructure.net>,\r
  24         Notmuch Mail <notmuch@notmuchmail.org>\r
  25 Subject: Re: [PATCH 00/11] add recipients to search output\r
  26 In-Reply-To: <1345427570-26518-1-git-send-email-jrollins@finestructure.net>\r
  27 References: <1345427570-26518-1-git-send-email-jrollins@finestructure.net>\r
  28 User-Agent: Notmuch/0.14+8~gd6b499d (http://notmuchmail.org) Emacs/23.1.1\r
  29         (x86_64-redhat-linux-gnu)\r
  30 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  31         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  32         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  33 Date: Thu, 23 Aug 2012 10:21:33 +0300\r
  34 Message-ID: <m2boi2cbuq.fsf@guru.guru-group.fi>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain; charset=us-ascii\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.13\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41         <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Thu, 23 Aug 2012 07:21:25 -0000\r
  50 \r
  51 On Mon, Aug 20 2012, Jameson Graef Rollins wrote:\r
  52 \r
  53 > This series is an attempt to add thread recipients to the search\r
  54 > output.\r
  55 >\r
  56 > My personal overall goal of this series is to support the handling of\r
  57 > drafts in the emacs ui.  For drafts we want to see recipients, instead\r
  58 > of authors, in the search output.  I can imagine other uses for this\r
  59 > series as well, though.\r
  60 >\r
  61 > The first four patches generalize the author list handling in thread\r
  62 > objects to handle any address list.  These patches could be applied\r
  63 > regardless of if the rest of the series is accepted.\r
  64 >\r
  65 > After that we modify the thread constructor such that it can hold\r
  66 > thread recipients as well.  Since there is overhead in retrieving\r
  67 > thread recipients from the message files (recipients are not stored in\r
  68 > the database) this is handled with a switch.\r
  69 >\r
  70 > Further patches add the new switch to the search CLI that adds thread\r
  71 > recipients to the structured output formats.  I didn't modify the text\r
  72 > output format, since there is no way to extend it.  I can imagine\r
  73 > tweaking the text output such that the author field is instead\r
  74 > replaced by the recipients (as is done for the emacs UI at the end of\r
  75 > the series), but that's not done here.\r
  76 >\r
  77 > In the emacs UI, I add a new toggle function that will toggle display\r
  78 > of thread authors or recipients in the 'authors' field of the search\r
  79 > output.  It's unfortunate that this ambiguity in that field name\r
  80 > remains, but I didn't know how to change that cleanly.  I'm working on\r
  81 > some tests for the new emacs functionality that I'll include in the\r
  82 > inevitable v2 of this series.\r
  83 \r
  84 I did not read much of this introduction before browsing to the code, I\r
  85 was about to comment whether attempt yo do less trivial tests are\r
  86 to be done.\r
  87 \r
  88 > The last patch is mostly just a tickle to suggest adding the\r
  89 > recipients to the database.  It would make the --include-recipient\r
  90 > searches much faster of course, but it might be overhead in the\r
  91 > database that folks aren't interested in.\r
  92 \r
  93 I got tickled... adding To (and Cc?!) to the database would also give\r
  94 (future notmuch?) address completion more addresses to match for.\r
  95 \r
  96 We should discuss whether to add other headers too? IIRC someone (Austin?)\r
  97 mentioned that everything (except Received:) headers could be there ?\r
  98 \r
  99 > As always, feedback, review, and comments are much appreciated.\r
 100 \r
 101 Overall, the code looks good (to me).\r
 102 \r
 103 > jamie.\r
 104 \r
 105 Tomi\r