f6/4975c26352d0005adf96120ad76e4ccf7f0d36

   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 65A0E431FC7\r
   6         for <notmuch@notmuchmail.org>; Mon, 29 Sep 2014 09:05:27 -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 PCmNDRzGIVxx for <notmuch@notmuchmail.org>;\r
  16         Mon, 29 Sep 2014 09:05:19 -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 A0ABD431FCF\r
  19         for <notmuch@notmuchmail.org>; Mon, 29 Sep 2014 09:05:18 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id E39F5100063;\r
  22         Mon, 29 Sep 2014 19:04:59 +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: <87k34rtoi2.fsf@steelpick.2x.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         <m27g0rpl6x.fsf@guru.guru-group.fi>\r
  30         <87k34rtoi2.fsf@steelpick.2x.cz>\r
  31 User-Agent: Notmuch/0.18.1+119~g5d90feb (http://notmuchmail.org) Emacs/24.3.1\r
  32         (x86_64-unknown-linux-gnu)\r
  33 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  34         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  35         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  36 Date: Mon, 29 Sep 2014 19:04:59 +0300\r
  37 Message-ID: <m27g0m1kgk.fsf@guru.guru-group.fi>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: text/plain\r
  40 X-BeenThere: notmuch@notmuchmail.org\r
  41 X-Mailman-Version: 2.1.13\r
  42 Precedence: list\r
  43 List-Id: "Use and development of the notmuch mail system."\r
  44         <notmuch.notmuchmail.org>\r
  45 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  46         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  47 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  48 List-Post: <mailto:notmuch@notmuchmail.org>\r
  49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  50 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  51         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  52 X-List-Received-Date: Mon, 29 Sep 2014 16:05:27 -0000\r
  53 \r
  54 On Thu, Sep 25 2014, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  55 \r
  56 > On Thu, Sep 25 2014, Tomi Ollila wrote:\r
  57 >> On Mon, Sep 22 2014, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  58 >>\r
  59 >>> Many functions that implement the search command need to access command\r
  60 >>> line options. Instead of passing each option in a separate variable, put\r
  61 >>> them in a structure and pass only this structure.\r
  62 >>\r
  63 >> This patch looks good to me.\r
  64 >\r
  65 > Thanks for the review.\r
  66 >\r
  67 >> Although the test and the implementation in the next patches look OK, I'd\r
  68 >> prefer the FLAG implementation Jani suggested earlier. IMO now that I\r
  69 >> compare these two it looks cleaner and simpler...\r
  70 >\r
  71 > The question is which kind of simplicity you have in mind. I think that\r
  72 > my version is simpler to type (less keystrokes). But if others have\r
  73 > different opinion, I don't mind.\r
  74 \r
  75 Less keystrokes for sure -- but these interfaces are usually accessed\r
  76 programmatically... :D\r
  77 \r
  78 >>\r
  79 >> Tomi\r
  80 >>\r
  81 >> (*) IMO the default unique (when requested) would be exact case-sensitive\r
  82 >> match of full name & address \r
  83 >\r
  84 > Why do you think that case-sensitive address matching should be the\r
  85 > default? In theory local-part can be case sensitive, but I've never seen\r
  86 > that in reality. So this default would only be useful if you want to\r
  87 > research how people type your email address :)\r
  88 \r
  89 Well, in short, I think the lowest level of uniqueness should be simple\r
  90 string match, and this should at least be available if not default --\r
  91 to the extent gmime provides (maybe that is this way in your patch...),\r
  92 \r
  93 ...and therefore I'd like to have this address output solved first, then\r
  94 we can experiment with the outputs provided and have better-educated\r
  95 comments on this issue...\r
  96 \r
  97 >> parts (phrase, address & comment); \r
  98 >\r
  99 > What do you mean by phrase and comment? Address syntax is defined by\r
 100 > http://tools.ietf.org/html/rfc5322#section-3.4.1.\r
 101 \r
 102 in \r
 103 \r
 104 "Foo Bar" (company/city) foo.bar@example.org\r
 105 \r
 106 and\r
 107 \r
 108 "Foo Bar" foo.bar@example.org (company/city) \r
 109 \r
 110 Phrase would be "Foo Bar"\r
 111 Address foo.bar@example.org\r
 112 and comment (company/city)\r
 113 \r
 114 As a side note, nottoomuch-addresses does some heuristics there, and think\r
 115 the 2 options above are equal (as "Phrase" (comment) address) -- which\r
 116 might the same InternetAddressMailbox provides :O\r
 117 \r
 118 Also, it seems that nottoomuch-addresses lowercases 'address' for\r
 119 comparison and storage ... I am not entirely sure whether I should provide\r
 120 options to disable these heuristics -- if someone asks for the feature\r
 121 then I probably will do :D\r
 122 \r
 123 >> then (a subset of possible) options could be:\r
 124 >>    +) case-insensitive (first match taken (or last match?) -- option?)\r
 125 >>    +) unique email addresses (take phrase/comment from first/last?)\r
 126 >>       --  or use first that has something additional to plain address\r
 127 >>       --  or use last  that has something additional to plain address\r
 128 >\r
 129 > Yes, there is a lot of possible options. I don't think that notmuch has\r
 130 > to support all of them. If people need something special like "use last\r
 131 > that has something additional to plain address", they can always do\r
 132 > --unique=none and do their own post-processing.\r
 133 \r
 134 Ok, but something (we can further bikeshed with) needs to be selected :D\r
 135 \r
 136 >\r
 137 > -Michal\r
 138 \r
 139 Tomi\r