d9/87b024fa5b0cc9e2d247c5c85aca0136d4e541

   1 Return-Path: <tomi.ollila@iki.fi>\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 53028431FB6\r
   6         for <notmuch@notmuchmail.org>; Sun,  2 Sep 2012 09:00:53 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id xEbL1FDvkx8a for <notmuch@notmuchmail.org>;\r
  16         Sun,  2 Sep 2012 09:00:52 -0700 (PDT)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 47D55431FAF\r
  19         for <notmuch@notmuchmail.org>; Sun,  2 Sep 2012 09:00:52 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id C12521000E5;\r
  22         Sun,  2 Sep 2012 19:00:58 +0300 (EEST)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH v2 0/5] emacs: notmuch-hello status message refresh hook,\r
  26         etc\r
  27 In-Reply-To: <cover.1346596156.git.jani@nikula.org>\r
  28 References: <cover.1346596156.git.jani@nikula.org>\r
  29 User-Agent: Notmuch/0.14+11~gd9bf007 (http://notmuchmail.org) Emacs/24.2.1\r
  30         (x86_64-unknown-linux-gnu)\r
  31 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  32         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  33         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  34 Date: Sun, 02 Sep 2012 19:00:58 +0300\r
  35 Message-ID: <m2sjb0wh1h.fsf@guru.guru-group.fi>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Sun, 02 Sep 2012 16:00:53 -0000\r
  51 \r
  52 On Sun, Sep 02 2012, Jani Nikula <jani@nikula.org> wrote:\r
  53 \r
  54 > Hi all, v2 of [1] addressing review comments: added NEWS, improved\r
  55 > documentation, made the query customizable.\r
  56 \r
  57 Hi Jani\r
  58 \r
  59 Just today I was looking these older patches below and was about to\r
  60 comment those when these arrived.\r
  61 \r
  62 I think the idea is great, but there is just something in the integration...\r
  63 \r
  64 > [1] id:"37899e28dbf67e4620a53279a869be3174c02d6f.1339775602.git.jani@nikula.org"\r
  65 \r
  66 I see hooks a great way to extend some functionality outside of that\r
  67 package instead of touching it. Also, maybe I've been using emacs too\r
  68 much as these hooks as defcustom seems little weird. Yet as I grepped\r
  69 'defcustom.*hook' through emacs-24.2/lisp/*.el and I got many hits --\r
  70 and some of those even added some ":options" to those (but most those\r
  71 I looked did not specify anything to choose from).\r
  72 \r
  73 That said, I am not _personally_ very thilled about the idea of defining\r
  74 functions inside notmuch and then that work via a hook...\r
  75 \r
  76 Therefore I'd like to propose an alternative way to do this:\r
  77 \r
  78 Integrate this change status information to the notmuch-hello-insert-header ()\r
  79 function, disabled by default -- and add customization variable which\r
  80 can be used to enable it. like:\r
  81 \r
  82  Welcome to notmuch. You have 22 195 messages.\r
  83 \r
  84  Welcome to notmuch. You have 22 185 messages (-10).\r
  85  Welcome to notmuch. You have 22 205 messages (+20).\r
  86 \r
  87 or second line for the status info.\r
  88 \r
  89 \r
  90 I would enable this feature if implemented this way. Also I would enable\r
  91 this implemented like this patch series if this is to be applied (I just \r
  92 give +-0). But this could also be implemented as a hook outside of notmuch\r
  93 relying that variable no-display is available in dynamic scope -- but without\r
  94 promising the variable will be there.\r
  95 \r
  96 > BR,\r
  97 > Jani.\r
  98 \r
  99 Finally, I hope my comment does not silence the audience but that this\r
 100 sparks more comment and good solution is used for this useful feature.\r
 101 \r
 102 Tomi\r
 103 \r
 104 >\r
 105 > Jani Nikula (5):\r
 106 >   emacs: document the notmuch-hello no-display argument\r
 107 >   emacs: add no-display arg to notmuch-hello-refresh-hook\r
 108 >   test: fix hook-counter to accept the new no-display param\r
 109 >   emacs: add notmuch hello refresh hook to display message count change\r
 110 >   NEWS: notmuch-hello refresh hook changes\r
 111 >\r
 112 >  NEWS                   |   18 ++++++++++++++++++\r
 113 >  emacs/notmuch-hello.el |   46 ++++++++++++++++++++++++++++++++++++++++++++--\r
 114 >  test/test-lib.el       |    2 +-\r
 115 >  3 files changed, 63 insertions(+), 3 deletions(-)\r
 116 >\r
 117 > -- \r
 118 > 1.7.9.5\r
 119 >\r
 120 > _______________________________________________\r
 121 > notmuch mailing list\r
 122 > notmuch@notmuchmail.org\r
 123 > http://notmuchmail.org/mailman/listinfo/notmuch\r