a2/821d12d247426c9c1922eb0e849ac8f4cd7630

   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 D3BF3431FB6\r
   6         for <notmuch@notmuchmail.org>; Sat, 20 Oct 2012 03:04:20 -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 dHTXeMhI9sBk for <notmuch@notmuchmail.org>;\r
  16         Sat, 20 Oct 2012 03:04:20 -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 6372B431FAE\r
  19         for <notmuch@notmuchmail.org>; Sat, 20 Oct 2012 03:04:20 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id 1CB5E1000E5;\r
  22         Sat, 20 Oct 2012 13:04:23 +0300 (EEST)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Ethan Glasser-Camp <ethan.glasser.camp@gmail.com>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH 7/8] lib: add NOTMUCH_EXCLUDE_FLAG to notmuch_exclude_t\r
  26 In-Reply-To: <87mwzi6tr6.fsf@betacantrips.com>\r
  27 References: <1340198947-29370-1-git-send-email-novalazy@gmail.com>\r
  28         <1340198947-29370-8-git-send-email-novalazy@gmail.com>\r
  29         <87txtr6o0c.fsf@betacantrips.com> <87mwzi6tr6.fsf@betacantrips.com>\r
  30 User-Agent: Notmuch/0.14+51~g62cd13b (http://notmuchmail.org) Emacs/24.2.1\r
  31         (x86_64-unknown-linux-gnu)\r
  32 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  33         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  34         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  35 Date: Sat, 20 Oct 2012 13:04:23 +0300\r
  36 Message-ID: <m28vb1a28o.fsf@guru.guru-group.fi>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Sat, 20 Oct 2012 10:04:21 -0000\r
  52 \r
  53 On Sat, Oct 20 2012, Ethan Glasser-Camp wrote:\r
  54 \r
  55 > Ethan Glasser-Camp <ethan.glasser.camp@gmail.com> writes:\r
  56 >\r
  57 >> It looks like you have better wording for patch 4/8 so I'd like to see\r
  58 >> you resend it.\r
  59 >>\r
  60 >> I'm marking patches 3, 4, and 7 as moreinfo. Please resubmit!\r
  61 >\r
  62 > It turns out that patch 4 already has a v2 in the thread, but I didn't\r
  63 > see it due to some kind of selective blindness. It would be nice if\r
  64 > nmbug had grouped it as part of the same patch series. \r
  65 \r
  66 I noticed the same today when I tagged your NEWS patch.\r
  67 \r
  68 By looking contrib/nmbug/nmbug-status the reason seems obvious:\r
  69 it iterates over messages and resets thread when thread id changes\r
  70 (and message ordering is not based on threads).\r
  71 \r
  72 There is 2 ways to "fix" this: \r
  73 \r
  74 1) Iterate over threads and then messges over these threads. I looked\r
  75 the bindings and this didn't seem so easy as one could imagine..\r
  76 \r
  77 2) Iterate over messages like now, but cache the content to an objects\r
  78 in hash where thread-id is the key.\r
  79  \r
  80 If no-one points me an easy way to do (1), I'll attempt (2) (in a distant\r
  81 future ;)\r
  82 \r
  83 >\r
  84 > I'm marking the old patch 4/8 obsolete. Only patch 3 and 7 remain.\r
  85 >\r
  86 > Ethan\r
  87 \r
  88 Tomi\r