c8/24fbc8a85599dfc1365f06fd2c8d11fbf6cfe3

   1 Return-Path: <aclements@csail.mit.edu>\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 C002F431FC9\r
   6         for <notmuch@notmuchmail.org>; Fri, 24 Oct 2014 05:40: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.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] 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 wjv3750uwI7q for <notmuch@notmuchmail.org>;\r
  16         Fri, 24 Oct 2014 05:40:22 -0700 (PDT)\r
  17 Received: from outgoing.csail.mit.edu (outgoing.csail.mit.edu [128.30.2.149])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 35D1D431FBD\r
  19         for <notmuch@notmuchmail.org>; Fri, 24 Oct 2014 05:40:22 -0700 (PDT)\r
  20 Received: from [104.131.20.129] (helo=awakeningjr)\r
  21         by outgoing.csail.mit.edu with esmtpsa (TLS1.0:RSA_AES_128_CBC_SHA1:16)\r
  22         (Exim 4.72) (envelope-from <aclements@csail.mit.edu>)\r
  23         id 1XheAD-0006Vv-LH; Fri, 24 Oct 2014 08:40:17 -0400\r
  24 Received: from amthrax by awakeningjr with local (Exim 4.84)\r
  25         (envelope-from <aclements@csail.mit.edu>)\r
  26         id 1XheAD-0001RW-2X; Fri, 24 Oct 2014 08:40:17 -0400\r
  27 Date: Fri, 24 Oct 2014 08:40:16 -0400\r
  28 From: Austin Clements <aclements@csail.mit.edu>\r
  29 To: David Edmondson <dme@dme.org>\r
  30 Subject: Re: [PATCH v1 1/3] search: Seperately report matching and\r
  31         non-matching    authors.\r
  32 Message-ID: <20141024124016.GG7970@csail.mit.edu>\r
  33 References: <1414140488-29978-1-git-send-email-dme@dme.org>\r
  34         <1414140488-29978-2-git-send-email-dme@dme.org>\r
  35         <87y4s5vn12.fsf@qmul.ac.uk>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain; charset=us-ascii\r
  38 Content-Disposition: inline\r
  39 In-Reply-To: <87y4s5vn12.fsf@qmul.ac.uk>\r
  40 User-Agent: Mutt/1.5.23 (2014-03-12)\r
  41 Cc: notmuch@notmuchmail.org\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.13\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Fri, 24 Oct 2014 12:40:29 -0000\r
  55 \r
  56 Quoth Mark Walters on Oct 24 at 10:23 am:\r
  57 > \r
  58 > Hi\r
  59 > \r
  60 > I definitely like the idea: some comments below.\r
  61 \r
  62 Agreed.\r
  63 \r
  64 > On Fri, 24 Oct 2014, David Edmondson <dme@dme.org> wrote:\r
  65 > > In addition to the :authors attribute of each search result, include\r
  66 > > :authors_matched and :authors_non_matched attributes. Both attributes\r
  67 > > are always included. If there are no non-matching authors, the\r
  68 > > :authors_non_matched attribute is set to the empty string.\r
  69 > \r
  70 > What about having both authors_matched and authors_not_matched as lists\r
  71 > of authors (ie one string for each author)? Then emacs, for example,\r
  72 > wouldn't try and parse the string back into authors before\r
  73 > splitting. And authors_non_matched could be an empty list when\r
  74 > appropriate which seems more natural than the empty string.\r
  75 > \r
  76 > All the above is based on what a client might want in the output rather\r
  77 > than what is easy or sensible to implement in the C code.\r
  78 \r
  79 I was going to suggest exactly the same thing.\r
  80 \r
  81 And I think there's a fairly easy way to do it in C code that will\r
  82 also prevent library interface bloat: instead of introducing new\r
  83 library APIs to get at this information, just use the existing\r
  84 notmuch_thread_get_messages API and construct the matched and\r
  85 non-matched lists in the CLI.  Doing it in the CLI wouldn't require\r
  86 the library to export yet another string list structure, which is\r
  87 always a huge pain (thanks C!), and wouldn't introduce more "helper"\r
  88 functions into the library API.\r
  89 \r
  90 I think the only disadvantage to doing this would be some duplication\r
  91 of the {matched_,}authors_{array,hash} logic into the CLI, but those\r
  92 are only there to support notmuch_thread_get_authors, which I think is\r
  93 a huge hack and should go away in the long term.  (And, by doing this\r
  94 list building in the CLI, we avoid further embellishing the\r
  95 unnecessary "get thread authors" API).\r