41/73d8bf2c218bee05a4a7adf72409673c55a9c9

   1 Return-Path: <jani@nikula.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 3E9CA431E64\r
   6         for <notmuch@notmuchmail.org>; Mon,  9 Jul 2012 22:10:11 -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.699\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.699 tagged_above=-999 required=5\r
  12         tests=[HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7] 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 DaPdwjlbrO8V for <notmuch@notmuchmail.org>;\r
  16         Mon,  9 Jul 2012 22:10:10 -0700 (PDT)\r
  17 Received: from mail-ob0-f181.google.com (mail-ob0-f181.google.com\r
  18         [209.85.214.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 84D1E431FBF\r
  21         for <notmuch@notmuchmail.org>; Mon,  9 Jul 2012 22:10:10 -0700 (PDT)\r
  22 Received: by obbup19 with SMTP id up19so17448987obb.26\r
  23         for <notmuch@notmuchmail.org>; Mon, 09 Jul 2012 22:10:10 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=mime-version:in-reply-to:references:date:message-id:subject:from:to\r
  27         :cc:content-type:x-gm-message-state;\r
  28         bh=Fg2Uwhh5zA8nEdMi2pcJI2a18tVrMZvYYmn8oBVGHnM=;\r
  29         b=CLzFdsI/6mzwIOdahpHC3Nru60PSAuB4eqi+GZogwO9HPQe70z3Zl+9Be+TFZIECab\r
  30         KLGdxvghbmewMZ3Oo/EABeWwTqSyHvx8udJv65d4WdgNeM7Bdf56Sf0S/bNmphoKkXh4\r
  31         LNUUlwXGlfWkAWVOQCozvYHUKu3NJDu/GGr8Mj2NEbNHXV1gtghmLeoyifwgmnWWdKGG\r
  32         gv3IXsEG/PmWxsCwm4WDBeQ7t6SHzVxEbIh3dbqnDGFR/P7HXi/iM5c4wURodkOgIJ6T\r
  33         9GlRj0YUEWrIG2TpeELAWXcqJOx/uD1xeRj+7aIopOvNGxHq7RP+FoalmOKYydJUNAIo\r
  34         vJzw==\r
  35 MIME-Version: 1.0\r
  36 Received: by 10.60.29.169 with SMTP id l9mr45156010oeh.14.1341897009938; Mon,\r
  37         09 Jul 2012 22:10:09 -0700 (PDT)\r
  38 Received: by 10.76.10.102 with HTTP; Mon, 9 Jul 2012 22:10:09 -0700 (PDT)\r
  39 Received: by 10.76.10.102 with HTTP; Mon, 9 Jul 2012 22:10:09 -0700 (PDT)\r
  40 In-Reply-To: <20120710014946.GB7332@mit.edu>\r
  41 References:\r
  42  <37899e28dbf67e4620a53279a869be3174c02d6f.1339775602.git.jani@nikula.org>\r
  43         <20120710014946.GB7332@mit.edu>\r
  44 Date: Tue, 10 Jul 2012 08:10:09 +0300\r
  45 Message-ID:\r
  46  <CAB+hUn8W_x_BNbzmWuyzReX-oaP67OK0t8y-yNZ5cKfRNdwOmg@mail.gmail.com>\r
  47 Subject: Re: [PATCH 1/3] emacs: add no-display arg to\r
  48         notmuch-hello-refresh-hook\r
  49 From: Jani Nikula <jani@nikula.org>\r
  50 To: Austin Clements <amdragon@mit.edu>\r
  51 Content-Type: multipart/alternative; boundary=e89a8ff256669a3fba04c472beb9\r
  52 X-Gm-Message-State:\r
  53  ALoCoQn99NemvpcM0NwaszaY+zElx/HEYIgy/U6R0zI6Qxi4q4+Cy98ageesncHD47kcMsYveDRn\r
  54 Cc: notmuch@notmuchmail.org\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Tue, 10 Jul 2012 05:10:11 -0000\r
  68 \r
  69 --e89a8ff256669a3fba04c472beb9\r
  70 Content-Type: text/plain; charset=UTF-8\r
  71 \r
  72 On Jul 10, 2012 4:49 AM, "Austin Clements" <amdragon@mit.edu> wrote:\r
  73 >\r
  74 > Quoth Jani Nikula on Jun 15 at  6:53 pm:\r
  75 > > Add no-display arg to notmuch-hello-refresh-hook to allow each hook to\r
  76 > > decide what is appropriate when no-display is t, which is typically\r
  77 > > the case when called non-interactively. This is used by the following\r
  78 > > patch.\r
  79 > >\r
  80 > > This breaks existing hooks people might have, which will now need to\r
  81 > > accept the argument.\r
  82 > >\r
  83 > > Signed-off-by: Jani Nikula <jani@nikula.org>\r
  84 >\r
  85 > This seems like an overloaded use of no-display.  If I'm reading the\r
  86 > code right, no-display indicates whether or not the notmuch-hello\r
  87 > buffer should be switched to and seems like a workaround for some\r
  88 > particular corner-case (I'm not even sure what).  This seems like a\r
  89 > strange condition to predicate a hook on (but maybe I just don't\r
  90 > understand).  What condition, abstractly speaking, is\r
  91 > notmuch-hello-refresh-status-message trying to run under?\r
  92 \r
  93 IIUC, no-display is useful for calling refresh from outside of emacs, e.g.\r
  94 from post-new hook in an automated fashion, so you can have an up-to-date\r
  95 buffer when you switch to it. There's no point in displaying the refresh\r
  96 message when you don't also switch to the buffer, is there? And this way\r
  97 you'll get the diff between the manual (through user interaction) refreshes\r
  98 of the buffer, not between two cron jobs.\r
  99 \r
 100 J.\r
 101 \r
 102 --e89a8ff256669a3fba04c472beb9\r
 103 Content-Type: text/html; charset=UTF-8\r
 104 Content-Transfer-Encoding: quoted-printable\r
 105 \r
 106 <p><br>\r
 107 On Jul 10, 2012 4:49 AM, &quot;Austin Clements&quot; &lt;<a href=3D"mailto:=\r
 108 amdragon@mit.edu">amdragon@mit.edu</a>&gt; wrote:<br>\r
 109 &gt;<br>\r
 110 &gt; Quoth Jani Nikula on Jun 15 at =C2=A06:53 pm:<br>\r
 111 &gt; &gt; Add no-display arg to notmuch-hello-refresh-hook to allow each ho=\r
 112 ok to<br>\r
 113 &gt; &gt; decide what is appropriate when no-display is t, which is typical=\r
 114 ly<br>\r
 115 &gt; &gt; the case when called non-interactively. This is used by the follo=\r
 116 wing<br>\r
 117 &gt; &gt; patch.<br>\r
 118 &gt; &gt;<br>\r
 119 &gt; &gt; This breaks existing hooks people might have, which will now need=\r
 120  to<br>\r
 121 &gt; &gt; accept the argument.<br>\r
 122 &gt; &gt;<br>\r
 123 &gt; &gt; Signed-off-by: Jani Nikula &lt;<a href=3D"mailto:jani@nikula.org"=\r
 124 >jani@nikula.org</a>&gt;<br>\r
 125 &gt;<br>\r
 126 &gt; This seems like an overloaded use of no-display. =C2=A0If I&#39;m read=\r
 127 ing the<br>\r
 128 &gt; code right, no-display indicates whether or not the notmuch-hello<br>\r
 129 &gt; buffer should be switched to and seems like a workaround for some<br>\r
 130 &gt; particular corner-case (I&#39;m not even sure what). =C2=A0This seems =\r
 131 like a<br>\r
 132 &gt; strange condition to predicate a hook on (but maybe I just don&#39;t<b=\r
 133 r>\r
 134 &gt; understand). =C2=A0What condition, abstractly speaking, is<br>\r
 135 &gt; notmuch-hello-refresh-status-message trying to run under?</p>\r
 136 <p>IIUC, no-display is useful for calling refresh from outside of emacs, e.=\r
 137 g. from post-new hook in an automated fashion, so you can have an up-to-dat=\r
 138 e buffer when you switch to it. There&#39;s no point in displaying the refr=\r
 139 esh message when you don&#39;t also switch to the buffer, is there? And thi=\r
 140 s way you&#39;ll get the diff between the manual (through user interaction)=\r
 141  refreshes of the buffer, not between two cron jobs.</p>\r
 142 \r
 143 <p>J.</p>\r
 144 \r
 145 --e89a8ff256669a3fba04c472beb9--\r