07/49278f6514948522fffd552611221c0c61d3a9

   1 Return-Path: <sojkam1@fel.cvut.cz>\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 5FC16431FAF\r
   6         for <notmuch@notmuchmail.org>; Wed,  5 Sep 2012 14:08:27 -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: -2.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] 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 C+hZfBsIjx0j for <notmuch@notmuchmail.org>;\r
  16         Wed,  5 Sep 2012 14:08:26 -0700 (PDT)\r
  17 Received: from max.feld.cvut.cz (max.feld.cvut.cz [147.32.192.36])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 22608431FAE\r
  19         for <notmuch@notmuchmail.org>; Wed,  5 Sep 2012 14:08:26 -0700 (PDT)\r
  20 Received: from localhost (unknown [192.168.200.4])\r
  21         by max.feld.cvut.cz (Postfix) with ESMTP id 8FA2C19F3375;\r
  22         Wed,  5 Sep 2012 23:08:20 +0200 (CEST)\r
  23 X-Virus-Scanned: IMAP AMAVIS\r
  24 Received: from max.feld.cvut.cz ([192.168.200.1])\r
  25         by localhost (styx.feld.cvut.cz [192.168.200.4]) (amavisd-new,\r
  26         port 10044)\r
  27         with ESMTP id yCeIwZZ819CH; Wed,  5 Sep 2012 23:08:19 +0200 (CEST)\r
  28 Received: from imap.feld.cvut.cz (imap.feld.cvut.cz [147.32.192.34])\r
  29         by max.feld.cvut.cz (Postfix) with ESMTP id 3E46A19F3367;\r
  30         Wed,  5 Sep 2012 23:08:14 +0200 (CEST)\r
  31 Received: from steelpick.2x.cz (unknown [213.29.198.144])\r
  32         (Authenticated sender: sojkam1)\r
  33         by imap.feld.cvut.cz (Postfix) with ESMTPSA id 4D39F660904;\r
  34         Wed,  5 Sep 2012 23:08:12 +0200 (CEST)\r
  35 Received: from wsh by steelpick.2x.cz with local (Exim 4.80)\r
  36         (envelope-from <sojkam1@fel.cvut.cz>)\r
  37         id 1T9MpX-000675-QZ; Wed, 05 Sep 2012 23:08:11 +0200\r
  38 From: Michal Sojka <sojkam1@fel.cvut.cz>\r
  39 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  40 Subject: Re: [PATCH v2 0/5] emacs: notmuch-hello status message refresh hook,\r
  41         etc\r
  42 In-Reply-To: <877gs9msy8.fsf@nikula.org>\r
  43 References: <cover.1346596156.git.jani@nikula.org>\r
  44         <87vcftbtmw.fsf@steelpick.2x.cz> <877gs9msy8.fsf@nikula.org>\r
  45 User-Agent: Notmuch/0.14+3~g608c52f (http://notmuchmail.org) Emacs/24.1.1\r
  46         (x86_64-pc-linux-gnu)\r
  47 Date: Wed, 05 Sep 2012 23:08:11 +0200\r
  48 Message-ID: <87zk54yy84.fsf@steelpick.2x.cz>\r
  49 MIME-Version: 1.0\r
  50 Content-Type: text/plain\r
  51 X-BeenThere: notmuch@notmuchmail.org\r
  52 X-Mailman-Version: 2.1.13\r
  53 Precedence: list\r
  54 List-Id: "Use and development of the notmuch mail system."\r
  55         <notmuch.notmuchmail.org>\r
  56 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  58 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  59 List-Post: <mailto:notmuch@notmuchmail.org>\r
  60 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  61 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  63 X-List-Received-Date: Wed, 05 Sep 2012 21:08:27 -0000\r
  64 \r
  65 On Tue, Sep 04 2012, Jani Nikula wrote:\r
  66 > On Tue, 04 Sep 2012, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  67 >> On Sun, Sep 02 2012, Jani Nikula wrote:\r
  68 >>> Hi all, v2 of [1] addressing review comments: added NEWS, improved\r
  69 >>> documentation, made the query customizable.\r
  70 >>>\r
  71 >>> [1]\r
  72 >>>     id:"37899e28dbf67e4620a53279a869be3174c02d6f.1339775602.git.jani@nikula.org"\r
  73 >>\r
  74 >> Hello Jani,\r
  75 >>\r
  76 >> if I understand correctly, the goal of this patchset is to display the\r
  77 >> message with the difference in the number of messages before and after\r
  78 >> refresh of notmuch-hello.\r
  79 >\r
  80 > Difference between two refreshes of notmuch-hello to be specific.\r
  81 >\r
  82 >> I think the current implementation is unnecessarily complicated. It\r
  83 >> would be sufficient to implement this directly in `notmuch-hello'\r
  84 >> without any hook. If `notmuch-hello-refresh-status-query' is nil\r
  85 >> (default) no message would be shown. If it is configured to something\r
  86 >> else, the message would be shown provided that no-display is nil. This\r
  87 >> way you don't have to break existing user's hooks and achieve the same\r
  88 >> behavior, don't you?\r
  89 >\r
  90 > I think it would be useful to pass no-display to hooks anyway,\r
  91 > regardless of the use here.\r
  92 \r
  93 I'm strongly against breaking existing user's setups without a good\r
  94 reason. The meaning of no-display parameter, as I understand it, is only\r
  95 to tells whether the refresh should happen in background or foreground.\r
  96 Isn't there any other method for hooks to check for this conditions? For\r
  97 example something like checking whether notmuch-hello buffer is shown in\r
  98 a window of the active frame? Or a simple (perhaps buffer-local)\r
  99 variable can contain this information.\r
 100 \r
 101 I don't know what people use notmuch-hello-refresh-hook for. I suppose\r
 102 that your use case (displaying a message outside of hello buffer) is\r
 103 rather an exception than a typical use.\r
 104 \r
 105 > I don't see that as a big reason to do this one way or the other.\r
 106 > Having this implemented directly in notmuch-hello does not make this\r
 107 > less complicated either.\r
 108 \r
 109 That's true, but it would be simpler for users to set it up. With your\r
 110 patches, users would have to configure the variable (query) as well as\r
 111 the hook. I propose to have only one place to configure this feature.\r
 112 \r
 113 -Michal\r