[PATCH] emacs: tree/show remove duplicate function

2014-07-16 Thread David Bremner
Mark Walters writes: > tree overrides notmuch-show-get-prop so that it can use many of the > utility function directly. Now that tree is in mainline the version > from tree can be moved to show and the original overridden show > version dropped. > --- Pushed. I hope there will be a followup patc

Re: [PATCH] emacs: tree/show remove duplicate function

2014-07-16 Thread David Bremner
Mark Walters writes: > tree overrides notmuch-show-get-prop so that it can use many of the > utility function directly. Now that tree is in mainline the version > from tree can be moved to show and the original overridden show > version dropped. > --- Pushed. I hope there will be a followup patc

[PATCH] emacs: tree/show remove duplicate function

2014-07-15 Thread David Bremner
Mark Walters writes: > On Tue, 15 Jul 2014, David Bremner wrote: >> Mark Walters writes: >>> + (cond ((eq major-mode 'notmuch-show-mode) >>> + (notmuch-show-get-message-properties)) >>> +((eq major-mode 'notmuch-tree-mode) >>> +

Re: [PATCH] emacs: tree/show remove duplicate function

2014-07-15 Thread David Bremner
Mark Walters writes: > On Tue, 15 Jul 2014, David Bremner wrote: >> Mark Walters writes: >>> + (cond ((eq major-mode 'notmuch-show-mode) >>> + (notmuch-show-get-message-properties)) >>> +((eq major-mode 'notmuch-tree-mode) >>> +

[PATCH] emacs: tree/show remove duplicate function

2014-07-15 Thread Mark Walters
On Tue, 15 Jul 2014, David Bremner wrote: > Mark Walters writes: >> + (cond ((eq major-mode 'notmuch-show-mode) >> + (notmuch-show-get-message-properties)) >> + ((eq major-mode 'notmuch-tree-mode) >> + (notmuch-tree-get-m

[PATCH] emacs: tree/show remove duplicate function

2014-07-15 Thread David Bremner
Mark Walters writes: > > Plausibly a comment and an explicit nil case would be clearer for the other > modes. > Your call. d

Re: [PATCH] emacs: tree/show remove duplicate function

2014-07-15 Thread David Bremner
Mark Walters writes: > > Plausibly a comment and an explicit nil case would be clearer for the other > modes. > Your call. d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH] emacs: tree/show remove duplicate function

2014-07-15 Thread Mark Walters
On Tue, 15 Jul 2014, David Bremner wrote: > Mark Walters writes: >> + (cond ((eq major-mode 'notmuch-show-mode) >> + (notmuch-show-get-message-properties)) >> + ((eq major-mode 'notmuch-tree-mode) >> + (notmuch-tree-get-m

[PATCH] emacs: tree/show remove duplicate function

2014-07-14 Thread David Bremner
Mark Walters writes: > +(cond ((eq major-mode 'notmuch-show-mode) > + (notmuch-show-get-message-properties)) > + ((eq major-mode 'notmuch-tree-mode) > + (notmuch-tree-get-message-properties I see this already exi

Re: [PATCH] emacs: tree/show remove duplicate function

2014-07-14 Thread David Bremner
Mark Walters writes: > +(cond ((eq major-mode 'notmuch-show-mode) > + (notmuch-show-get-message-properties)) > + ((eq major-mode 'notmuch-tree-mode) > + (notmuch-tree-get-message-properties I see this already exi

[PATCH] emacs: tree/show remove duplicate function

2014-06-07 Thread Tomi Ollila
On Sat, Jun 07 2014, Mark Walters wrote: > tree overrides notmuch-show-get-prop so that it can use many of the > utility function directly. Now that tree is in mainline the version > from tree can be moved to show and the original overridden show > version dropped. > --- > I should have done this

[PATCH] emacs: tree/show remove duplicate function

2014-06-07 Thread Mark Walters
tree overrides notmuch-show-get-prop so that it can use many of the utility function directly. Now that tree is in mainline the version from tree can be moved to show and the original overridden show version dropped. --- I should have done this ages ago but forgot about it. Best wishes Mark em

Re: [PATCH] emacs: tree/show remove duplicate function

2014-06-07 Thread Tomi Ollila
On Sat, Jun 07 2014, Mark Walters wrote: > tree overrides notmuch-show-get-prop so that it can use many of the > utility function directly. Now that tree is in mainline the version > from tree can be moved to show and the original overridden show > version dropped. > --- > I should have done this

[PATCH] emacs: tree/show remove duplicate function

2014-06-07 Thread Mark Walters
tree overrides notmuch-show-get-prop so that it can use many of the utility function directly. Now that tree is in mainline the version from tree can be moved to show and the original overridden show version dropped. --- I should have done this ages ago but forgot about it. Best wishes Mark em