9a/6bcb55b5a3d6d4f4b76f2ba268b784a2bf19da

   1 Return-Path: <cworth@cworth.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 59BC3431FBF;\r
   6         Wed, 18 Nov 2009 01:42:17 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id j-W1SQlJ7v4j; Wed, 18 Nov 2009 01:42:16 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 6C6B8431FAE;\r
  13         Wed, 18 Nov 2009 01:42:16 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: Keith Packard <keithp@keithp.com>, Mikhail Gusarov\r
  16         <dottedmag@dottedmag.net>, notmuch@notmuchmail.org\r
  17 In-Reply-To: <yunbpj0etua.fsf@aiko.keithp.com>\r
  18 References: <1258471718-6781-1-git-send-email-dottedmag@dottedmag.net>\r
  19         <87lji5cbwo.fsf@yoom.home.cworth.org> <yunbpj0etua.fsf@aiko.keithp.com>\r
  20 Date: Wed, 18 Nov 2009 01:42:02 -0800\r
  21 Message-ID: <87pr7gqidx.fsf@yoom.home.cworth.org>\r
  22 MIME-Version: 1.0\r
  23 Content-Type: text/plain; charset=us-ascii\r
  24 Subject: Re: [notmuch] [PATCH 1/2] Close message file after parsing message\r
  25  headers\r
  26 X-BeenThere: notmuch@notmuchmail.org\r
  27 X-Mailman-Version: 2.1.12\r
  28 Precedence: list\r
  29 List-Id: "Use and development of the notmuch mail system."\r
  30         <notmuch.notmuchmail.org>\r
  31 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  32         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  33 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  34 List-Post: <mailto:notmuch@notmuchmail.org>\r
  35 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  36 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  37         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  38 X-List-Received-Date: Wed, 18 Nov 2009 09:42:17 -0000\r
  39 \r
  40 On Tue, 17 Nov 2009 13:15:25 -0800, Keith Packard <keithp@keithp.com> wrote:\r
  41 > Threading the message also involves displaying the from and to contents,\r
  42 > which requires opening the message file. The alternative to the fix I\r
  43 > provided is to just parse all of the message headers when first opening\r
  44 > the message; it could then be immediately closed and the hash referred\r
  45 > to for all header data. Given the choice, just having the caller say\r
  46 > when it has finished with a message is probably a reasonable option...\r
  47 \r
  48 Hi Keith,\r
  49 \r
  50 Once I finally got back on the ground again, I pushed out a revised\r
  51 version of your patch, (didn't need the reply-to stuff anymore since I\r
  52 had fixed that differently in the meantime).\r
  53 \r
  54 I'm pretty happy with the state of this portion of the code now.\r
  55 \r
  56 Thanks Keith and Mikhail for your input on and code to fix this bug.\r
  57 \r
  58 -Carl\r
  59 \r