97/668bf29dc27aa9e043c12101a4e00aa3f085a2

   1 Return-Path: <BATV+28bf489b39c899822e75+2435+infradead.org+hohndel@bombadil.srs.infradead.org>\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 97F314196F3\r
   6         for <notmuch@notmuchmail.org>; Sat, 24 Apr 2010 12:09:58 -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.301\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.301 tagged_above=-999 required=5\r
  12         tests=[BAYES_20=-0.001, RCVD_IN_DNSWL_MED=-2.3] autolearn=ham\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 D3aGUKD0-0aT for <notmuch@notmuchmail.org>;\r
  16         Sat, 24 Apr 2010 12:09:57 -0700 (PDT)\r
  17 Received: from bombadil.infradead.org (bombadil.infradead.org [18.85.46.34])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 976AC431FC1\r
  19         for <notmuch@notmuchmail.org>; Sat, 24 Apr 2010 12:09:57 -0700 (PDT)\r
  20 Received: from localhost ([::1] helo=x200.gr8dns.org)\r
  21         by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux))\r
  22         id 1O5kjk-00035e-Uf; Sat, 24 Apr 2010 19:09:57 +0000\r
  23 Received: by x200.gr8dns.org (Postfix, from userid 500)\r
  24         id 4A412CC5AC; Sat, 24 Apr 2010 12:09:56 -0700 (PDT)\r
  25 From: Dirk Hohndel <hohndel@infradead.org>\r
  26 To: Carl Worth <cworth@cworth.org>, notmuch@notmuchmail.org\r
  27 Subject: Re: improve from-header guessing\r
  28 In-Reply-To: <87pr1qf1lj.fsf@yoom.home.cworth.org>\r
  29 References: <1271451102-11336-1-git-send-email-hohndel@infradead.org>\r
  30         <87pr1qf1lj.fsf@yoom.home.cworth.org>\r
  31 User-Agent: Notmuch/0.2-191-gdcd7da3 (http://notmuchmail.org) Emacs/23.1.1\r
  32         (i386-redhat-linux-gnu)\r
  33 Date: Sat, 24 Apr 2010 12:09:56 -0700\r
  34 Message-ID: <m3633gsm4b.fsf@x200.gr8dns.org>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain; charset=us-ascii\r
  37 X-SRS-Rewrite: SMTP reverse-path rewritten from <hohndel@infradead.org> by\r
  38         bombadil.infradead.org See http://www.infradead.org/rpr.html\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, 24 Apr 2010 19:09:58 -0000\r
  52 \r
  53 On Fri, 23 Apr 2010 11:47:04 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  54 > On Fri, 16 Apr 2010 13:51:40 -0700, Dirk Hohndel <hohndel@infradead.org> wrote:\r
  55 > > The following two patches should address most of the concerns raised \r
  56 > > to my previous series. \r
  57 > \r
  58 > Allow me to raise new concerns then. ;-)\r
  59 \r
  60 Any time\r
  61  \r
  62 > > The first patch simply adds an interface to obtain a concatenation of\r
  63 > > all instances of a specific header from an email.\r
  64 > \r
  65 > I was hoping to see the "special-case value of NULL" go away with this\r
  66 > change.\r
  67 > \r
  68 > And I like that there's a new function to get the concatenated header,\r
  69 > (I would prefer an unabbreviated name of get_concatenated_header than\r
  70 > get_header_concat), but I don't like seeing all the existing callers of\r
  71 > get_header updated to pass an extra 0. Instead, I'd prefer to see those\r
  72 > calls unchanged, and a tiny new get_header that passes the 0 and then\r
  73 > make the actual implementing function be static and named something like\r
  74 > notmuch_message_file_get_header_internal.\r
  75 \r
  76 Turns out that the way I did this was broken anyway. So we can simply\r
  77 forget these patches and your concerns. I'm sure you'll raise new\r
  78 concerns on the new ("rearchitected") patches.\r
  79 \r
  80 > Both patches have some trailing whitespace. I see these easily wince I\r
  81 > have the following in my ~/.gitconfig:\r
  82 > \r
  83 >       [core]\r
  84 >               whitespace = trailing-space,space-before-tab\r
  85 \r
  86 I know. I'm trying to be better about checking whitespace pollution\r
  87 before submitting things.\r
  88 \r
  89 > Finally, I'd like to see some tests for this feature. (But we do have\r
  90 > the feature already without tests, so I won't strictly block on that).\r
  91 \r
  92 Hu? You even commited these already. Or am I reading email out of order\r
  93 again? \r
  94 \r
  95 /D\r