[PATCH v2] emacs: Don't override mm-show-part in notmuch-show-view-part

2013-05-26 Thread David Bremner
Austin Clements writes: > > This patch removes the mm-show-part override in favor of passing the > file name in to mm-show-part the way it expects, so we get its default Pushed, d

[PATCH v2] emacs: Don't override mm-show-part in notmuch-show-view-part

2013-05-26 Thread Tomi Ollila
On Sun, May 26 2013, Austin Clements wrote: > Previously, notmuch-show-view-part overrode the function binding of > mm-show-part to redirect it to notmuch-show-save-part to get notmuch's > default file name handling in case mm-display-part decided to fall > back to saving the part. In addition

[PATCH v2] emacs: Don't override mm-show-part in notmuch-show-view-part

2013-05-26 Thread Mark Walters
LGTM +1 This does not rely on the followup Austin mentioned in id:20130526063627.GQ5999 at mit.edu is so I think this should go in as is as it fixes the flet thing without changing the behaviour. Best wishes Mark On Sun, 26 May 2013, Austin Clements wrote: > Previously,

[PATCH v2] emacs: Don't override mm-show-part in notmuch-show-view-part

2013-05-26 Thread Austin Clements
Previously, notmuch-show-view-part overrode the function binding of mm-show-part to redirect it to notmuch-show-save-part to get notmuch's default file name handling in case mm-display-part decided to fall back to saving the part. In addition to being messy, this depended on the now-deprecated

[PATCH v2] emacs: Don't override mm-show-part in notmuch-show-view-part

2013-05-26 Thread Austin Clements
Previously, notmuch-show-view-part overrode the function binding of mm-show-part to redirect it to notmuch-show-save-part to get notmuch's default file name handling in case mm-display-part decided to fall back to saving the part. In addition to being messy, this depended on the now-deprecated

Re: [PATCH v2] emacs: Don't override mm-show-part in notmuch-show-view-part

2013-05-26 Thread Mark Walters
LGTM +1 This does not rely on the followup Austin mentioned in id:20130526063627.gq5...@mit.edu is so I think this should go in as is as it fixes the flet thing without changing the behaviour. Best wishes Mark On Sun, 26 May 2013, Austin Clements amdra...@mit.edu wrote: Previously,

Re: [PATCH v2] emacs: Don't override mm-show-part in notmuch-show-view-part

2013-05-26 Thread Tomi Ollila
On Sun, May 26 2013, Austin Clements amdra...@mit.edu wrote: Previously, notmuch-show-view-part overrode the function binding of mm-show-part to redirect it to notmuch-show-save-part to get notmuch's default file name handling in case mm-display-part decided to fall back to saving the part.

Re: [PATCH v2] emacs: Don't override mm-show-part in notmuch-show-view-part

2013-05-26 Thread David Bremner
Austin Clements amdra...@mit.edu writes: This patch removes the mm-show-part override in favor of passing the file name in to mm-show-part the way it expects, so we get its default Pushed, d ___ notmuch mailing list notmuch@notmuchmail.org