git.notmuchmail.org Git - notmuch/blob - test/corpora/default/foo/baz/11:2,

   1 From: "Keith Packard" <keithp@keithp.com>
   2 To: notmuch@notmuchmail.org
   3 Date: Tue, 17 Nov 2009 13:15:25 -0800
   4 Subject: [notmuch] [PATCH 1/2] Close message file after parsing message
   5  headers
   6 In-Reply-To: <87lji5cbwo.fsf@yoom.home.cworth.org>
   7 References: <1258471718-6781-1-git-send-email-dottedmag@dottedmag.net>
   8         <87lji5cbwo.fsf@yoom.home.cworth.org>
   9 Message-ID: <yunbpj0etua.fsf@aiko.keithp.com>
  10
  11 On Tue, 17 Nov 2009 09:13:27 -0800, Carl Worth <cworth at cworth.org> wrote:
  12
  13 > I didn't apply Keith's fix yet, because I think I'd rather just fix the
  14 > indexer to store the In-Reply-To header in a separate term prefix from
  15 > the term used for the References header[*]. That will then let us lookup
  16 > the in-reply-to value later for thread constructions without having to
  17 > open the original email file at all.
  18
  19 Threading the message also involves displaying the from and to contents,
  20 which requires opening the message file. The alternative to the fix I
  21 provided is to just parse all of the message headers when first opening
  22 the message; it could then be immediately closed and the hash referred
  23 to for all header data. Given the choice, just having the caller say
  24 when it has finished with a message is probably a reasonable option...
  25
  26 -keith
  27