73/dc1a87f6dc4e5a469f20a5d5814208b0843b55

   1 Return-Path: <cworth@cworth.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 55D06431FAF\r
   6         for <notmuch@notmuchmail.org>; Wed, 27 Jun 2012 09:45:03 -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 kn58lznsirsy for <notmuch@notmuchmail.org>;\r
  16         Wed, 27 Jun 2012 09:45:02 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id A3EFD431FB6\r
  21         for <notmuch@notmuchmail.org>; Wed, 27 Jun 2012 09:45:02 -0700 (PDT)\r
  22 Received: from yoom.amr.corp.intel.com (localhost [127.0.0.1])\r
  23         by arlo.cworth.org (Postfix) with ESMTP id 1FEF929A053;\r
  24         Wed, 27 Jun 2012 09:45:01 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: notmuch@notmuchmail.org\r
  27 Subject: [PATCH] Restore original keybinding ('r' = reply-to-all)\r
  28 Date: Wed, 27 Jun 2012 09:46:05 -0700\r
  29 Message-Id: <1340815565-21083-1-git-send-email-cworth@cworth.org>\r
  30 X-Mailer: git-send-email 1.7.10\r
  31 X-BeenThere: notmuch@notmuchmail.org\r
  32 X-Mailman-Version: 2.1.13\r
  33 Precedence: list\r
  34 List-Id: "Use and development of the notmuch mail system."\r
  35         <notmuch.notmuchmail.org>\r
  36 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  37         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  38 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  39 List-Post: <mailto:notmuch@notmuchmail.org>\r
  40 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  41 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  42         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  43 X-List-Received-Date: Wed, 27 Jun 2012 16:45:03 -0000\r
  44 \r
  45 Since the beginning of time, the emacs interface provided a keybinding\r
  46 of 'r' to reply to a message, (and originally, all recipients).\r
  47 \r
  48 Then, before release 0.12 the emacs interface acquired a new\r
  49 reply-to-sender only feature. In commit\r
  50 f02b475fa781bb5df3358c73213e7633a99f016e the new feature was put onto\r
  51 the original keybinding, (and reply-to-all was moved to 'R').\r
  52 \r
  53 This restores the original keybinding and uses the new keybinding for\r
  54 the new feature.\r
  55 ---\r
  56 \r
  57 The commit message above motivates this change from the point of view\r
  58 of interface compatibility. That argument would have been stronger if\r
  59 I had made it in January, (at the time of the change), rather than\r
  60 now, (since there's now a new incompatiblity made by changing things\r
  61 back).\r
  62 \r
  63 That said, I still think the compatibility argument is sound.\r
  64 \r
  65 I have other feelings as to which reply behavior should be treated\r
  66 preferentially. But those are potentially more controversial, so I'll\r
  67 leave the bikeshedding out of our commit history.\r
  68 \r
  69 Thanks to everyone keeping notmuch alive and well during my extended\r
  70 absence from maintainership. What a tremendous free-software success\r
  71 story this all is. I'm so grateful to all of you.\r
  72 \r
  73 -Carl\r
  74 \r
  75  emacs/notmuch-show.el |    4 ++--\r
  76  emacs/notmuch.el      |    4 ++--\r
  77  2 files changed, 4 insertions(+), 4 deletions(-)\r
  78 \r
  79 diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el\r
  80 index 4349836..399a8bd 100644\r
  81 --- a/emacs/notmuch-show.el\r
  82 +++ b/emacs/notmuch-show.el\r
  83 @@ -1145,8 +1145,8 @@ reset based on the original query."\r
  84         (define-key map "s" 'notmuch-search)\r
  85         (define-key map "m" 'notmuch-mua-new-mail)\r
  86         (define-key map "f" 'notmuch-show-forward-message)\r
  87 -       (define-key map "r" 'notmuch-show-reply-sender)\r
  88 -       (define-key map "R" 'notmuch-show-reply)\r
  89 +       (define-key map "r" 'notmuch-show-reply)\r
  90 +       (define-key map "R" 'notmuch-show-reply-sender)\r
  91         (define-key map "|" 'notmuch-show-pipe-message)\r
  92         (define-key map "w" 'notmuch-show-save-attachments)\r
  93         (define-key map "V" 'notmuch-show-view-raw-message)\r
  94 diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
  95 index c6236db..3a6b025 100644\r
  96 --- a/emacs/notmuch.el\r
  97 +++ b/emacs/notmuch.el\r
  98 @@ -215,8 +215,8 @@ For a mouse binding, return nil."\r
  99      (define-key map ">" 'notmuch-search-last-thread)\r
 100      (define-key map "p" 'notmuch-search-previous-thread)\r
 101      (define-key map "n" 'notmuch-search-next-thread)\r
 102 -    (define-key map "r" 'notmuch-search-reply-to-thread-sender)\r
 103 -    (define-key map "R" 'notmuch-search-reply-to-thread)\r
 104 +    (define-key map "r" 'notmuch-search-reply-to-thread)\r
 105 +    (define-key map "R" 'notmuch-search-reply-to-thread-sender)\r
 106      (define-key map "m" 'notmuch-mua-new-mail)\r
 107      (define-key map "s" 'notmuch-search)\r
 108      (define-key map "o" 'notmuch-search-toggle-order)\r
 109 -- \r
 110 1.7.10\r
 111 \r