1a/619da4754b0846e3890e5476ef9eac0c5a7b1c

   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 34178431FC0;\r
   6         Wed, 18 Nov 2009 15:44:22 -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 W3K2jjCxsG4p; Wed, 18 Nov 2009 15:44:21 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 3F0E7431FBC;\r
  13         Wed, 18 Nov 2009 15:44:21 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>, notmuch\r
  16         <notmuch@notmuchmail.org>\r
  17 In-Reply-To: <87my2jrhjh.fsf@linux.vnet.ibm.com>\r
  18 References: <87my2jrhjh.fsf@linux.vnet.ibm.com>\r
  19 Date: Thu, 19 Nov 2009 00:44:07 +0100\r
  20 Message-ID: <877htn760o.fsf@yoom.home.cworth.org>\r
  21 MIME-Version: 1.0\r
  22 Content-Type: text/plain; charset=us-ascii\r
  23 Subject: Re: [notmuch] [PATCH] notmuch: Add hooks for show-mode and\r
  24  search-mode\r
  25 X-BeenThere: notmuch@notmuchmail.org\r
  26 X-Mailman-Version: 2.1.12\r
  27 Precedence: list\r
  28 List-Id: "Use and development of the notmuch mail system."\r
  29         <notmuch.notmuchmail.org>\r
  30 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  31         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  32 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  33 List-Post: <mailto:notmuch@notmuchmail.org>\r
  34 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  35 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  36         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  37 X-List-Received-Date: Wed, 18 Nov 2009 23:44:22 -0000\r
  38 \r
  39 On Wed, 18 Nov 2009 20:44:58 +0530, aneesh.kumar@linux.vnet.ibm.com (Aneesh Kumar K.V) wrote:\r
  40 > From: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>\r
  41 > Date: Wed, 18 Nov 2009 20:43:13 +0530\r
  42 > Subject: [PATCH] notmuch: Add hooks for show-mode and search-mode\r
  43 > \r
  44 > This enables to do things like\r
  45 > \r
  46 > (add-hook 'notmuch-search-mode-hook\r
  47 >           (lambda()\r
  48 >             (hl-line-mode 1) ))\r
  49 \r
  50 Hi Aneesh,\r
  51 \r
  52 Thanks for the patch. I think the hooks will be genuinely useful. I\r
  53 haven't pushed this yet for two questions:\r
  54 \r
  55 1. I recently merged an independent patch from Keith to add one hook\r
  56 already. (And it looks like Keith got fancy and used defcustom instead\r
  57 of defvar.) Could you re-do your patch on top of master and match the\r
  58 style he used? [*]\r
  59 \r
  60 2. If this hl-line-mode thing is interesting for you, might it not be\r
  61 interesting for all users? (What does it do anyway?)\r
  62 \r
  63 One concern I have with hooks is that people will use them to fix things\r
  64 locally and prevent the defaults getting fixed to improve things for\r
  65 everybody. (Granted, sometimes there's an honest difference of opinion,\r
  66 so the defaults can't always satisfy everyone, and we'll want the\r
  67 hooks.)\r
  68 \r
  69 I guess I'm just hoping that people will at least attempt to get\r
  70 improvements to be integrated into the defaults instead of just changing\r
  71 local hooks.\r
  72 \r
  73 -Carl\r
  74 \r
  75 [*] Independently, it might be nice to switch all of our variables\r
  76 intended to be tweaked by the user from defvar to defcustom.\r