91/1adc9fe688f560b5a3039756aab2b7604e9689

   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 C1A17431FAF\r
   6         for <notmuch@notmuchmail.org>; Tue, 17 Jul 2012 02:31:06 -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 r3oDuFW+Wo6q for <notmuch@notmuchmail.org>;\r
  16         Tue, 17 Jul 2012 02:31:06 -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 E9748431FAE\r
  19         for <notmuch@notmuchmail.org>; Tue, 17 Jul 2012 02:31:05 -0700 (PDT)\r
  20 Received: by guru.guru-group.fi (Postfix, from userid 501)\r
  21         id 569BD10014D; Tue, 17 Jul 2012 12:31:15 +0300 (EEST)\r
  22 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  23 To: Mark Walters <markwalters1009@gmail.com>,\r
  24  Ingo Lohmar <i.lohmar@gmail.com>,      notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH] emacs: Fix notmuch-message-mark-replied.\r
  26 In-Reply-To: <87bojqrd6d.fsf@qmul.ac.uk>\r
  27 References: <87vcj8ckzm.fsf@acer.localhost.com> <87bojqrd6d.fsf@qmul.ac.uk>\r
  28 User-Agent: Notmuch/0.13.2+93~ge4fdd97 (http://notmuchmail.org) Emacs/23.1.1\r
  29         (x86_64-redhat-linux-gnu)\r
  30 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  31         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  32         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  33 Date: Tue, 17 Jul 2012 12:31:15 +0300\r
  34 Message-ID: <m2liii20po.fsf@guru.guru-group.fi>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain; charset=us-ascii\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.13\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41         <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Tue, 17 Jul 2012 09:31:06 -0000\r
  50 \r
  51 On Sun, Jul 08 2012, Mark Walters wrote:\r
  52 \r
  53 > On Sun, 03 Jun 2012, Ingo Lohmar <i.lohmar@gmail.com> wrote:\r
  54 >> notmuch-message-mark-replied used "apply" to change message tags\r
  55 >> according to notmuch-message-replied-tags after sending a reply.  This\r
  56 >> works if the latter is a single-element list.  But with the recently\r
  57 >> changed format of tag changes, it breaks for multiple-element lists.\r
  58 >> Use "funcall" to properly pass the list of tag changes as a single\r
  59 >> argument.\r
  60 >\r
  61 > This looks correct to me: the bug is still in current master and this\r
  62 > does fix it.\r
  63 >\r
  64 > As Jamie says, it would be nice to have a test. Unfortunately, that is\r
  65 > beyond my test/emacs skills. \r
  66 >\r
  67 > Note this bug does not occur with default configuration but the option\r
  68 > is a defcustom option, and following the example in that option's\r
  69 > documentation will cause the problem.\r
  70 >\r
  71 > Since this patch has been around for over a month I think it should\r
  72 > probably be applied.\r
  73 \r
  74 I just tested this, without the patch setting multiple tags in reply\r
  75 fails with pretty ugly error message. with the patch setting one or\r
  76 multiple tags works. \r
  77 The fix is trivial (and obvious if one looks at it).\r
  78 \r
  79 I remove the needs-review tag.\r
  80 \r
  81 >\r
  82 > Best wishes\r
  83 >\r
  84 > Mark\r
  85 \r
  86 Tomi\r
  87 \r
  88 >\r
  89 >> ---\r
  90 >>  emacs/notmuch-message.el |    2 +-\r
  91 >>  1 file changed, 1 insertion(+), 1 deletion(-)\r
  92 >>\r
  93 >> diff --git a/emacs/notmuch-message.el b/emacs/notmuch-message.el\r
  94 >> index 5964caa..d3738bf 100644\r
  95 >> --- a/emacs/notmuch-message.el\r
  96 >> +++ b/emacs/notmuch-message.el\r
  97 >> @@ -45,7 +45,7 @@ the \"inbox\" and \"todo\", you would set\r
  98 >>                              (concat "+" str)\r
  99 >>                            str))\r
 100 >>                        notmuch-message-replied-tags)))\r
 101 >> -    (apply 'notmuch-tag (notmuch-id-to-query (car (car rep))) tags)))))\r
 102 >> +    (funcall 'notmuch-tag (notmuch-id-to-query (car (car rep))) tags)))))\r
 103 >>  \r
 104 >>  (add-hook 'message-send-hook 'notmuch-message-mark-replied)\r
 105 >>  \r
 106 >> -- \r
 107 >> 1.7.10\r