[PATCH 2/2] emacs: Unbind M-RET as display of thread with crypto switch.

[David Bremner]

On Sat, 12 Nov 2011 13:12:31 -0800, Jameson Graef Rollins  wrote:
> So for some reason this patch and the three that follow it were not
> pushed when the first was.  They were all incorrectly marked in nmbug as
> being pushed well.  I assume this was just an oversight.  In any event,
> these should be pushed, since they normalize the behavior of this
> keybinding, and fix documentation.

Hi Jamie, I pushed two of these, but

   id:"1306695383-18651-1-git-send-email-jrollins at finestructure.net"
   id:"1306695383-18651-2-git-send-email-jrollins at finestructure.net"

did not apply for me, even with git am -3

d

Re: [PATCH 2/2] emacs: Unbind M-RET as display of thread with crypto switch.

[David Bremner]

On Sat, 12 Nov 2011 13:12:31 -0800, Jameson Graef Rollins 
 wrote:
> So for some reason this patch and the three that follow it were not
> pushed when the first was.  They were all incorrectly marked in nmbug as
> being pushed well.  I assume this was just an oversight.  In any event,
> these should be pushed, since they normalize the behavior of this
> keybinding, and fix documentation.

Hi Jamie, I pushed two of these, but

   id:"1306695383-18651-1-git-send-email-jroll...@finestructure.net"
   id:"1306695383-18651-2-git-send-email-jroll...@finestructure.net"

did not apply for me, even with git am -3

d
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH 2/2] emacs: Unbind M-RET as display of thread with crypto switch.

[Jameson Graef Rollins]

So for some reason this patch and the three that follow it were not
pushed when the first was.  They were all incorrectly marked in nmbug as
being pushed well.  I assume this was just an oversight.  In any event,
these should be pushed, since they normalize the behavior of this
keybinding, and fix documentation.

jamie.


pgpvZtuvXL7FX.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch

[PATCH 2/2] emacs: Unbind M-RET as display of thread with crypto switch.

[Jameson Graef Rollins]

So for some reason this patch and the three that follow it were not
pushed when the first was.  They were all incorrectly marked in nmbug as
being pushed well.  I assume this was just an oversight.  In any event,
these should be pushed, since they normalize the behavior of this
keybinding, and fix documentation.

jamie.
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: 

[PATCH 2/2] emacs: Unbind M-RET as display of thread with crypto switch.

[Jameson Graef Rollins]

Use prefix argument instead to set switch.
---
 emacs/notmuch.el |7 +--
 1 files changed, 1 insertions(+), 6 deletions(-)

diff --git a/emacs/notmuch.el b/emacs/notmuch.el
index 3311fe8..0978c66 100644
--- a/emacs/notmuch.el
+++ b/emacs/notmuch.el
@@ -218,7 +218,6 @@ For a mouse binding, return nil."
 (define-key map "-" 'notmuch-search-remove-tag)
 (define-key map "+" 'notmuch-search-add-tag)
 (define-key map (kbd "RET") 'notmuch-search-show-thread)
-(define-key map (kbd "M-RET") 'notmuch-search-show-thread-crypto-switch)
 map)
   "Keymap for \"notmuch search\" buffers.")
 (fset 'notmuch-search-mode-map notmuch-search-mode-map)
@@ -418,13 +417,9 @@ Complete list of currently available key bindings:
   "Return a list of authors for the current region"
   (notmuch-search-properties-in-region 'notmuch-search-subject beg end))
 
-(defun notmuch-search-show-thread-crypto-switch ()
-  (interactive)
-  (notmuch-search-show-thread t))
-
 (defun notmuch-search-show-thread (&optional crypto-switch)
   "Display the currently selected thread."
-  (interactive)
+  (interactive "P")
   (let ((thread-id (notmuch-search-find-thread-id))
(subject (notmuch-search-find-subject)))
 (if (> (length thread-id) 0)
-- 
1.7.4.4

___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch

[PATCH 2/2] emacs: Unbind M-RET as display of thread with crypto switch.

[Jameson Graef Rollins]

Use prefix argument instead to set switch.
---
 emacs/notmuch.el |7 +--
 1 files changed, 1 insertions(+), 6 deletions(-)

diff --git a/emacs/notmuch.el b/emacs/notmuch.el
index 3311fe8..0978c66 100644
--- a/emacs/notmuch.el
+++ b/emacs/notmuch.el
@@ -218,7 +218,6 @@ For a mouse binding, return nil."
 (define-key map "-" 'notmuch-search-remove-tag)
 (define-key map "+" 'notmuch-search-add-tag)
 (define-key map (kbd "RET") 'notmuch-search-show-thread)
-(define-key map (kbd "M-RET") 'notmuch-search-show-thread-crypto-switch)
 map)
   "Keymap for \"notmuch search\" buffers.")
 (fset 'notmuch-search-mode-map notmuch-search-mode-map)
@@ -418,13 +417,9 @@ Complete list of currently available key bindings:
   "Return a list of authors for the current region"
   (notmuch-search-properties-in-region 'notmuch-search-subject beg end))

-(defun notmuch-search-show-thread-crypto-switch ()
-  (interactive)
-  (notmuch-search-show-thread t))
-
 (defun notmuch-search-show-thread (&optional crypto-switch)
   "Display the currently selected thread."
-  (interactive)
+  (interactive "P")
   (let ((thread-id (notmuch-search-find-thread-id))
(subject (notmuch-search-find-subject)))
 (if (> (length thread-id) 0)
-- 
1.7.4.4