ce/fad69bdd089deb1b3ee828756b2329f994ce19

   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 7DECE431FB6\r
   6         for <notmuch@notmuchmail.org>; Sat, 18 Feb 2012 23:05:22 -0800 (PST)\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 v1EnDRINdeG1 for <notmuch@notmuchmail.org>;\r
  16         Sat, 18 Feb 2012 23:05:19 -0800 (PST)\r
  17 Received: from mail-pw0-f53.google.com (mail-pw0-f53.google.com\r
  18         [209.85.160.53]) (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 323FC431FAE\r
  21         for <notmuch@notmuchmail.org>; Sat, 18 Feb 2012 23:05:19 -0800 (PST)\r
  22 Received: by pbbrq13 with SMTP id rq13so6011632pbb.26\r
  23         for <notmuch@notmuchmail.org>; Sat, 18 Feb 2012 23:05:17 -0800 (PST)\r
  24 Received-SPF: pass (google.com: domain of jani@nikula.org designates\r
  25         10.68.225.73 as permitted sender) client-ip=10.68.225.73; \r
  26 Authentication-Results: mr.google.com;\r
  27         spf=pass (google.com: domain of jani@nikula.org\r
  28         designates 10.68.225.73 as permitted sender)\r
  29         smtp.mail=jani@nikula.org\r
  30 Received: from mr.google.com ([10.68.225.73])\r
  31         by 10.68.225.73 with SMTP id ri9mr48470454pbc.70.1329635117259\r
  32         (num_hops = 1); Sat, 18 Feb 2012 23:05:17 -0800 (PST)\r
  33 MIME-Version: 1.0\r
  34 Received: by 10.68.225.73 with SMTP id ri9mr39770025pbc.70.1329635117204; Sat,\r
  35         18 Feb 2012 23:05:17 -0800 (PST)\r
  36 Received: by 10.68.12.103 with HTTP; Sat, 18 Feb 2012 23:05:17 -0800 (PST)\r
  37 Received: by 10.68.12.103 with HTTP; Sat, 18 Feb 2012 23:05:17 -0800 (PST)\r
  38 In-Reply-To: <1329605059-15358-1-git-send-email-sojkam1@fel.cvut.cz>\r
  39 References: <1329605059-15358-1-git-send-email-sojkam1@fel.cvut.cz>\r
  40 Date: Sun, 19 Feb 2012 09:05:17 +0200\r
  41 Message-ID:\r
  42  <CAB+hUn907hwS46jM4FY+BAhBTE1VMaMs=q0PFEou3DfWWaDF8g@mail.gmail.com>\r
  43 Subject: Re: [PATCH] emacs: Mention race condition safety in user visible\r
  44         documentation\r
  45 From: Jani Nikula <jani@nikula.org>\r
  46 To: Michal Sojka <sojkam1@fel.cvut.cz>\r
  47 Content-Type: multipart/alternative; boundary=e89a8ff2437dd784aa04b94bccfb\r
  48 X-Gm-Message-State:\r
  49  ALoCoQkI+WQSKtHf8JyDkLiE2vnaFQV84ZVIpDuUwM6iRDEbmHNCbTW9106o+V56fUubL2KjrKoi\r
  50 Cc: notmuch@notmuchmail.org\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: Sun, 19 Feb 2012 07:05:22 -0000\r
  64 \r
  65 --e89a8ff2437dd784aa04b94bccfb\r
  66 Content-Type: text/plain; charset=UTF-8\r
  67 \r
  68 On Feb 19, 2012 12:44 AM, "Michal Sojka" <sojkam1@fel.cvut.cz> wrote:\r
  69 >\r
  70 > After recent rework of a/A/x/X key bindings, the important paragraph in\r
  71 > documentation of `notmuch-show-archive-thread' stayed hidden from users,\r
  72 > because no key is bound to this function.\r
  73 >\r
  74 > This patch copies the important paragraph to the documentation of\r
  75 > functions currently bound to keys.\r
  76 \r
  77 Hi Michal, this was discussed earlier, and Dmitry convinced me that the\r
  78 code does exactly what the user expects it to do (archives only the\r
  79 messages in the buffer) so it would be unnecessary and confusing to have\r
  80 this documentation. It could be added as a comment for developers I guess.\r
  81 \r
  82 BR,\r
  83 Jani.\r
  84 \r
  85 > ---\r
  86 >  emacs/notmuch-show.el |   14 ++++++++++++--\r
  87 >  1 files changed, 12 insertions(+), 2 deletions(-)\r
  88 >\r
  89 > diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el\r
  90 > index aa9ccee..1916146 100644\r
  91 > --- a/emacs/notmuch-show.el\r
  92 > +++ b/emacs/notmuch-show.el\r
  93 > @@ -1721,13 +1721,23 @@ buffer."\r
  94 >     (notmuch-show-tag-all (concat op "inbox"))))\r
  95 >\r
  96 >  (defun notmuch-show-archive-thread-then-next ()\r
  97 > -  "Archive each message in thread, then show next thread from search."\r
  98 > +  "Archive each message in thread, then show next thread from search.\r
  99 > +\r
 100 > +Note: This command is safe from any race condition of new messages\r
 101 > +being delivered to the same thread. It does not archive the\r
 102 > +entire thread, but only the messages shown in the current\r
 103 > +buffer."\r
 104 >   (interactive)\r
 105 >   (notmuch-show-archive-thread)\r
 106 >   (notmuch-show-next-thread t))\r
 107 >\r
 108 >  (defun notmuch-show-archive-thread-then-exit ()\r
 109 > -  "Archive each message in thread, then exit back to search results."\r
 110 > +  "Archive each message in thread, then exit back to search results.\r
 111 > +\r
 112 > +Note: This command is safe from any race condition of new messages\r
 113 > +being delivered to the same thread. It does not archive the\r
 114 > +entire thread, but only the messages shown in the current\r
 115 > +buffer."\r
 116 >   (interactive)\r
 117 >   (notmuch-show-archive-thread)\r
 118 >   (notmuch-show-next-thread))\r
 119 > --\r
 120 > 1.7.7.3\r
 121 >\r
 122 > _______________________________________________\r
 123 > notmuch mailing list\r
 124 > notmuch@notmuchmail.org\r
 125 > http://notmuchmail.org/mailman/listinfo/notmuch\r
 126 \r
 127 --e89a8ff2437dd784aa04b94bccfb\r
 128 Content-Type: text/html; charset=UTF-8\r
 129 Content-Transfer-Encoding: quoted-printable\r
 130 \r
 131 <p><br>\r
 132 On Feb 19, 2012 12:44 AM, &quot;Michal Sojka&quot; &lt;<a href=3D"mailto:so=\r
 133 jkam1@fel.cvut.cz">sojkam1@fel.cvut.cz</a>&gt; wrote:<br>\r
 134 &gt;<br>\r
 135 &gt; After recent rework of a/A/x/X key bindings, the important paragraph i=\r
 136 n<br>\r
 137 &gt; documentation of `notmuch-show-archive-thread&#39; stayed hidden from =\r
 138 users,<br>\r
 139 &gt; because no key is bound to this function.<br>\r
 140 &gt;<br>\r
 141 &gt; This patch copies the important paragraph to the documentation of<br>\r
 142 &gt; functions currently bound to keys.</p>\r
 143 <p>Hi Michal, this was discussed earlier, and Dmitry convinced me that the =\r
 144 code does exactly what the user expects it to do (archives only the message=\r
 145 s in the buffer) so it would be unnecessary and confusing to have this docu=\r
 146 mentation. It could be added as a comment for developers I guess.</p>\r
 147 \r
 148 <p>BR,<br>\r
 149 Jani.<br>\r
 150  </p>\r
 151 <p>&gt; ---<br>\r
 152 &gt; =C2=A0emacs/notmuch-show.el | =C2=A0 14 ++++++++++++--<br>\r
 153 &gt; =C2=A01 files changed, 12 insertions(+), 2 deletions(-)<br>\r
 154 &gt;<br>\r
 155 &gt; diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el<br>\r
 156 &gt; index aa9ccee..1916146 100644<br>\r
 157 &gt; --- a/emacs/notmuch-show.el<br>\r
 158 &gt; +++ b/emacs/notmuch-show.el<br>\r
 159 &gt; @@ -1721,13 +1721,23 @@ buffer.&quot;<br>\r
 160 &gt; =C2=A0 =C2=A0 (notmuch-show-tag-all (concat op &quot;inbox&quot;))))<b=\r
 161 r>\r
 162 &gt;<br>\r
 163 &gt; =C2=A0(defun notmuch-show-archive-thread-then-next ()<br>\r
 164 &gt; - =C2=A0&quot;Archive each message in thread, then show next thread fr=\r
 165 om search.&quot;<br>\r
 166 &gt; + =C2=A0&quot;Archive each message in thread, then show next thread fr=\r
 167 om search.<br>\r
 168 &gt; +<br>\r
 169 &gt; +Note: This command is safe from any race condition of new messages<br=\r
 170 >\r
 171 &gt; +being delivered to the same thread. It does not archive the<br>\r
 172 &gt; +entire thread, but only the messages shown in the current<br>\r
 173 &gt; +buffer.&quot;<br>\r
 174 &gt; =C2=A0 (interactive)<br>\r
 175 &gt; =C2=A0 (notmuch-show-archive-thread)<br>\r
 176 &gt; =C2=A0 (notmuch-show-next-thread t))<br>\r
 177 &gt;<br>\r
 178 &gt; =C2=A0(defun notmuch-show-archive-thread-then-exit ()<br>\r
 179 &gt; - =C2=A0&quot;Archive each message in thread, then exit back to search=\r
 180  results.&quot;<br>\r
 181 &gt; + =C2=A0&quot;Archive each message in thread, then exit back to search=\r
 182  results.<br>\r
 183 &gt; +<br>\r
 184 &gt; +Note: This command is safe from any race condition of new messages<br=\r
 185 >\r
 186 &gt; +being delivered to the same thread. It does not archive the<br>\r
 187 &gt; +entire thread, but only the messages shown in the current<br>\r
 188 &gt; +buffer.&quot;<br>\r
 189 &gt; =C2=A0 (interactive)<br>\r
 190 &gt; =C2=A0 (notmuch-show-archive-thread)<br>\r
 191 &gt; =C2=A0 (notmuch-show-next-thread))<br>\r
 192 &gt; --<br>\r
 193 &gt; 1.7.7.3<br>\r
 194 &gt;<br>\r
 195 &gt; _______________________________________________<br>\r
 196 &gt; notmuch mailing list<br>\r
 197 &gt; <a href=3D"mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.org</a>=\r
 198 <br>\r
 199 &gt; <a href=3D"http://notmuchmail.org/mailman/listinfo/notmuch">http://not=\r
 200 muchmail.org/mailman/listinfo/notmuch</a><br>\r
 201 </p>\r
 202 \r
 203 --e89a8ff2437dd784aa04b94bccfb--\r