git.notmuchmail.org Git - notmuch/blob - test/corpus/cur/07:2,

   1 From: "Carl Worth" <cworth@cworth.org>
   2 To: notmuch@notmuchmail.org
   3 Date: Tue, 17 Nov 2009 09:13:27 -0800
   4 Subject: [notmuch] [PATCH 1/2] Close message file after parsing message
   5  headers
   6 In-Reply-To: <1258471718-6781-1-git-send-email-dottedmag@dottedmag.net>
   7 References: <1258471718-6781-1-git-send-email-dottedmag@dottedmag.net>
   8 Message-ID: <87lji5cbwo.fsf@yoom.home.cworth.org>
   9
  10 On Tue, 17 Nov 2009 21:28:37 +0600, Mikhail Gusarov <dottedmag at dottedmag.net> wrote:
  11 > Keeping unused files open helps to see "Too many open files" often.
  12 >
  13 > Signed-off-by: Mikhail Gusarov <dottedmag at dottedmag.net>
  14 ...
  15 On Tue, 17 Nov 2009 21:28:38 +0600, Mikhail Gusarov <dottedmag at dottedmag.net> wrote:
  16 >
  17 > Signed-off-by: Mikhail Gusarov <dottedmag at dottedmag.net>
  18 > ---
  19 >  lib/message.cc |    2 ++
  20 >  1 files changed, 2 insertions(+), 0 deletions(-)
  21
  22 Hi Mikhail,
  23
  24 Welcome to notmuch, and thanks for these patches! I've pushed both of
  25 them out now.
  26
  27 Keith ran into the same problem of "too many open files" and wrote a
  28 more complex fix, (which included what you did here). His code can be
  29 seen at:
  30
  31         git://keithp.com/git/notmuch
  32
  33 I didn't apply Keith's fix yet, because I think I'd rather just fix the
  34 indexer to store the In-Reply-To header in a separate term prefix from
  35 the term used for the References header[*]. That will then let us lookup
  36 the in-reply-to value later for thread constructions without having to
  37 open the original email file at all.
  38
  39 -Carl
  40
  41 [*] Yes, this is my first post to our new mailing list and I'm already
  42 spouting off about "terms" and "prefixes" without any definitions. I
  43 apologize for that. I hope that people will ask questions freely here on
  44 the list whenever anything is not clear, and I'll be glad to explain
  45 things as needed. (Then when can shove answers into a HACKING document.)
  46
  47 PS. This reply is a great example of a feature that notmuch *almost*
  48 supports already---repling to multiple messages at once. The "notmuch
  49 reply" command line does everything necessary to make this work, but we
  50 haven't yet hooked up any keybindings for this in the emacs client yet.
  51 Obviously, 'r' from the search view could reply to the entire thread.
  52 But when viewing a thread, anyone have a good keybinding suggestion?
  53 (There's obviously 'R' as opposed to 'r', but I think we'll probably
  54 want to distinguish "reply to sender" from "reply to all" before trying
  55 to distinguish "reply to message" from "reply to thread" (which I
  56 imagine is more rare of an operation).
  57