ad/1580df080bf60e44f54ce0c25ea9d9ad105928

   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 2090E431FC0;\r
   6         Fri, 27 Nov 2009 17:15:51 -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 cPgnj0zDe6s8; Fri, 27 Nov 2009 17:15:50 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 429B7431FAE;\r
  13         Fri, 27 Nov 2009 17:15:50 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: Bart Trojanowski <bart@jukie.net>, notmuch@notmuchmail.org\r
  16 In-Reply-To: <1259124875-28212-3-git-send-email-bart@jukie.net>\r
  17 References: <1259124875-28212-1-git-send-email-bart@jukie.net>\r
  18         <1259124875-28212-2-git-send-email-bart@jukie.net>\r
  19         <1259124875-28212-3-git-send-email-bart@jukie.net>\r
  20 Date: Fri, 27 Nov 2009 17:15:35 -0800\r
  21 Message-ID: <87iqcvbgaw.fsf@yoom.home.cworth.org>\r
  22 MIME-Version: 1.0\r
  23 Content-Type: text/plain; charset=us-ascii\r
  24 Cc: Bart Trojanowski <bart@jukie.net>\r
  25 Subject: Re: [notmuch] [PATCH 2/3] have _notmuch_thread_create mark which\r
  26  messages matched the query\r
  27 X-BeenThere: notmuch@notmuchmail.org\r
  28 X-Mailman-Version: 2.1.12\r
  29 Precedence: list\r
  30 List-Id: "Use and development of the notmuch mail system."\r
  31         <notmuch.notmuchmail.org>\r
  32 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  33         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  34 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  35 List-Post: <mailto:notmuch@notmuchmail.org>\r
  36 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  37 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  38         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  39 X-List-Received-Date: Sat, 28 Nov 2009 01:15:51 -0000\r
  40 \r
  41 On Tue, 24 Nov 2009 23:54:34 -0500, Bart Trojanowski <bart@jukie.net> wrote:\r
  42 > When _notmuch_thread_create() is given a query string, it can return more\r
  43 > messages than just those matching the query.  To distinguish those that\r
  44 > matched the query expression, the MATCHING_SEARCH flag is set\r
  45 > appropriately.\r
  46 \r
  47 This is a very useful feature, Bart. Thanks for coding it up. And it's a\r
  48 nicely-implemented patch series as well.\r
  49 \r
  50 I've pushed this out now, so anyone reading along should be able to try\r
  51 it with:\r
  52 \r
  53 ./notmuch show thread:6d5e3e276461188c5778c9f219f63782 and subject:"PATCH 2/3"\r
  54 \r
  55 I can't wait to have the emacs support for opening only matched messages\r
  56 by default.\r
  57 \r
  58 By the way, do you think that this support obviates the\r
  59 --only-matching-messages option for "notmuch search" or does anyone\r
  60 still want that?\r
  61 \r
  62 Or maybe the right fix is to make "notmuch show" display only matching\r
  63 messages by default, (which will likely be more friendly to a user\r
  64 manually typing "notmuch show" at the command line). And then make the\r
  65 user-interfaces pass an "--entire-thread" option (or so) to get the\r
  66 current results.\r
  67 \r
  68 If we're going to make the command-line user-interface usable on its\r
  69 own, then I definitely want to make it be the user interfaces that have\r
  70 to pass extra-long command-line options to get what they want.\r
  71 \r
  72 One quick point on naming:\r
  73 \r
  74 >  /* Message flags */\r
  75 >  typedef enum _notmuch_message_flag {\r
  76 > +    NOTMUCH_MSG_FLAG_MATCHING_SEARCH,\r
  77 >  } notmuch_message_flag_t;\r
  78 \r
  79 I like my enum values to match their type name without abbreviation. I\r
  80 also like internals (like this enum value) to match the way they are\r
  81 exposed in the interface, (which in this case is "match"). So I'd like\r
  82 the above value to instead be:\r
  83 \r
  84         NOTMUCH_MESSAGE_FLAG_MATCH\r
  85 \r
  86 -Carl\r