72/4c52c28d271d2a141dd361fd51d9b59681dac2

   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 5EE19431FB6\r
   6         for <notmuch@notmuchmail.org>; Thu, 25 Sep 2014 12:14:14 -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 30aYcDaDyjkE for <notmuch@notmuchmail.org>;\r
  16         Thu, 25 Sep 2014 12:14:09 -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 BF6AB431FAF\r
  19         for <notmuch@notmuchmail.org>; Thu, 25 Sep 2014 12:14:09 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id 68279100033;\r
  22         Thu, 25 Sep 2014 22:13:58 +0300 (EEST)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Michal Sojka <sojkam1@fel.cvut.cz>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH 1/5] cli: Refactor option passing in the search command\r
  26 In-Reply-To: <1411378679-7307-2-git-send-email-sojkam1@fel.cvut.cz>\r
  27 References: <1411378679-7307-1-git-send-email-sojkam1@fel.cvut.cz>\r
  28         <1411378679-7307-2-git-send-email-sojkam1@fel.cvut.cz>\r
  29 User-Agent: Notmuch/0.18.1+97~g6c845ef (http://notmuchmail.org) Emacs/24.3.1\r
  30         (x86_64-unknown-linux-gnu)\r
  31 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  32         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  33         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  34 Date: Thu, 25 Sep 2014 22:13:58 +0300\r
  35 Message-ID: <m27g0rpl6x.fsf@guru.guru-group.fi>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Thu, 25 Sep 2014 19:14:14 -0000\r
  51 \r
  52 On Mon, Sep 22 2014, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  53 \r
  54 > Many functions that implement the search command need to access command\r
  55 > line options. Instead of passing each option in a separate variable, put\r
  56 > them in a structure and pass only this structure.\r
  57 \r
  58 This patch looks good to me. \r
  59 \r
  60 Although the test and the implementation in the next patches look OK, I'd\r
  61 prefer the FLAG implementation Jani suggested earlier. IMO now that I\r
  62 compare these two it looks cleaner and simpler...\r
  63 \r
  64 I.e. I'd prefer notmuch search --output=sender --output=recipients ...\r
  65 (same output regardless the order these options given).\r
  66 \r
  67 I'd postpone the unique handling to a bit later phase; there are quite a\r
  68 few options how to do that (*)\r
  69 \r
  70 \r
  71 Tomi\r
  72 \r
  73 (*) IMO the default unique (when requested) would be exact case-sensitive\r
  74 match of full name & address parts (phrase, address & comment); then\r
  75 (a subset of possible) options could be: \r
  76    +) case-insensitive (first match taken (or last match?) -- option?)\r
  77    +) unique email addresses (take phrase/comment from first/last?)\r
  78       --  or use first that has something additional to plain address\r
  79       --  or use last  that has something additional to plain address\r
  80 \r
  81 \r
  82 > This will become handy in the following patches.\r
  83 > ---\r
  84 >  notmuch-search.c | 122 ++++++++++++++++++++++++++++---------------------------\r
  85 >  1 file changed, 62 insertions(+), 60 deletions(-)\r
  86 >\r
  87 > diff --git a/notmuch-search.c b/notmuch-search.c\r
  88 > index bc9be45..5ac2a26 100644\r
  89 > --- a/notmuch-search.c\r
  90 > +++ b/notmuch-search.c\r