59/d6e9a7ae06a5a6d2377369c644c35ec7a885eb

   1 Return-Path: <m.walters@qmul.ac.uk>\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 DAB754086D1\r
   6         for <notmuch@notmuchmail.org>; Sat,  7 Jan 2012 03:57:45 -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: -1.098\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.098 tagged_above=-999 required=5\r
  12         tests=[DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001,\r
  13         NML_ADSP_CUSTOM_MED=1.2, RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id ZWokFhbzHOOt for <notmuch@notmuchmail.org>;\r
  17         Sat,  7 Jan 2012 03:57:45 -0800 (PST)\r
  18 Received: from mail2.qmul.ac.uk (mail2.qmul.ac.uk [138.37.6.6])\r
  19         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id 4A0704086CD\r
  22         for <notmuch@notmuchmail.org>; Sat,  7 Jan 2012 03:57:45 -0800 (PST)\r
  23 Received: from smtp.qmul.ac.uk ([138.37.6.40])\r
  24         by mail2.qmul.ac.uk with esmtp (Exim 4.71)\r
  25         (envelope-from <m.walters@qmul.ac.uk>)\r
  26         id 1RjUu4-0006zN-7O; Sat, 07 Jan 2012 11:57:40 +0000\r
  27 Received: from 94-192-233-223.zone6.bethere.co.uk ([94.192.233.223]\r
  28         helo=localhost)\r
  29         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.69)\r
  30         (envelope-from <m.walters@qmul.ac.uk>)\r
  31         id 1RjUu3-0004Hr-Th; Sat, 07 Jan 2012 11:57:40 +0000\r
  32 From: Mark Walters <markwalters1009@gmail.com>\r
  33 To: Jani Nikula <jani@nikula.org>, David Bremner <david@tethera.net>\r
  34 Subject: Re: [PATCH 1/4] cli: fix use of uninitialized variable in "notmuch\r
  35         reply"\r
  36 In-Reply-To:\r
  37  <CAB+hUn9cWzDYK7ZvKWvsU=cCNuU80ERR8gT+T_Tu=SfMXH7K5A@mail.gmail.com>\r
  38 References: <cover.1325794371.git.jani@nikula.org>\r
  39         <974a2ef1f1df7d93e0b5bd642ca8a49f8b727a86.1325794371.git.jani@nikula.org>\r
  40         <87sjjsjhjk.fsf@zancas.localnet>\r
  41         <CAB+hUn9cWzDYK7ZvKWvsU=cCNuU80ERR8gT+T_Tu=SfMXH7K5A@mail.gmail.com>\r
  42 User-Agent: Notmuch/0.10.2+183~g99cd7be (http://notmuchmail.org) Emacs/23.3.1\r
  43         (i486-pc-linux-gnu)\r
  44 Date: Sat, 07 Jan 2012 11:57:39 +0000\r
  45 Message-ID: <877h13u3m4.fsf@qmul.ac.uk>\r
  46 MIME-Version: 1.0\r
  47 Content-Type: text/plain; charset=us-ascii\r
  48 X-Sender-Host-Address: 94.192.233.223\r
  49 X-QM-SPAM-Info: Sender has good ham record.  :)\r
  50 X-QM-Body-MD5: 9b9883f281cdd8500f9de6e1ecf68b48 (of first 20000 bytes)\r
  51 X-SpamAssassin-Score: -1.7\r
  52 X-SpamAssassin-SpamBar: -\r
  53 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
  54         determine if it is\r
  55         spam. We require at least 5.0 points to mark a message as spam.\r
  56         This message scored -1.7 points.\r
  57         Summary of the scoring: \r
  58         * -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\r
  59         *      medium trust\r
  60         *      [138.37.6.40 listed in list.dnswl.org]\r
  61         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
  62         provider *      (markwalters1009[at]gmail.com)\r
  63         * -0.0 T_RP_MATCHES_RCVD Envelope sender domain matches handover relay\r
  64         *      domain\r
  65         *  0.6 AWL AWL: From: address is in the auto white-list\r
  66 X-QM-Scan-Virus: ClamAV says the message is clean\r
  67 Cc: notmuch@notmuchmail.org\r
  68 X-BeenThere: notmuch@notmuchmail.org\r
  69 X-Mailman-Version: 2.1.13\r
  70 Precedence: list\r
  71 List-Id: "Use and development of the notmuch mail system."\r
  72         <notmuch.notmuchmail.org>\r
  73 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  74         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  75 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  76 List-Post: <mailto:notmuch@notmuchmail.org>\r
  77 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  78 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  79         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  80 X-List-Received-Date: Sat, 07 Jan 2012 11:57:46 -0000\r
  81 \r
  82 \r
  83 Hello\r
  84 \r
  85 I do not get the failure with just 1/4 applied but do with all 4\r
  86 applied. The trivial patch below fixes it, but it might not be the best\r
  87 solution.\r
  88 \r
  89 The failure occurs because Jani's patch changes the behavior of a couple\r
  90 of emacs/notmuch internal functions: the function\r
  91 notmuch-search-reply-to-thread is renamed to\r
  92 notmuch-search-reply-all-to-thread and a new\r
  93 notmuch-search-reply-to-thread function is added (which does reply to\r
  94 sender) and a similar change for notmuch-show-reply-.. . Since the\r
  95 keybindings are also remapped the user will not notice any difference.\r
  96 \r
  97 However, if the user has any key-bindings etc in their .emacs file the\r
  98 behaviour could change. It might be preferable to keep the existing\r
  99 functions as they are and give the new reply-to-sender functions a new\r
 100 name.\r
 101 \r
 102 Best wishes\r
 103 \r
 104 Mark\r
 105 \r
 106 \r
 107 diff --git a/test/emacs b/test/emacs\r
 108 index a06c223..5047d46 100755\r
 109 --- a/test/emacs\r
 110 +++ b/test/emacs\r
 111 @@ -258,7 +258,7 @@ test_expect_equal_file OUTPUT EXPECTED\r
 112  test_begin_subtest "Reply within emacs"\r
 113  test_emacs '(notmuch-search "subject:\"testing message sent via SMTP\"")\r
 114             (notmuch-test-wait)\r
 115 -           (notmuch-search-reply-to-thread)\r
 116 +           (notmuch-search-reply-all-to-thread)\r
 117             (test-output)'\r
 118  sed -i -e 's/^In-Reply-To: <.*>$/In-Reply-To: <XXX>/' OUTPUT\r
 119  cat <<EOF >EXPECTED\r
 120 \r
 121 \r
 122 \r