Re: [PATCH v1 0/3] Improve the acquisition of text parts.

2016-03-27 Thread David Bremner
David Edmondson writes: > Improve the acquisition of text parts. > > This affects the new "reply" behaviour and the rendering of > application/octet-stream parts that are treated as text. > pushed. d ___ notmuch mailing list

Re: [PATCH v1 0/3] Improve the acquisition of text parts.

2016-03-26 Thread Mark Walters
Hi Sorry this email ended up rather long: Summary: I have run a test (see below) on all of the lkml part of the performance-corpus, and all the changes look expected. So this series looks good to me. First note how we do the bodypart-insertion: for a mime type of text/plain we first try the

Re: [PATCH v1 0/3] Improve the acquisition of text parts.

2016-03-14 Thread David Bremner
David Edmondson writes: > On Sun, Mar 13 2016, Mark Walters wrote: >> However, it would be sensible to get testing in a greater variety of >> charsets/encodings > > Agreed. Does anyone have suggestions on how we might achieve this? A > corpus of mail that we could use? Maybe the

Re: [PATCH v1 0/3] Improve the acquisition of text parts.

2016-03-14 Thread David Edmondson
On Sun, Mar 13 2016, Mark Walters wrote: > However, it would be sensible to get testing in a greater variety of > charsets/encodings Agreed. Does anyone have suggestions on how we might achieve this? A corpus of mail that we could use? ___ notmuch

Re: [PATCH v1 0/3] Improve the acquisition of text parts.

2016-03-13 Thread Mark Walters
This looks good to me +1. However, it would be sensible to get testing in a greater variety of charsets/encodings Best wishes Mark On Tue, 08 Mar 2016, David Edmondson wrote: > Improve the acquisition of text parts. > > This affects the new "reply" behaviour and the rendering

[PATCH v1 0/3] Improve the acquisition of text parts.

2016-03-08 Thread David Edmondson
Improve the acquisition of text parts. This affects the new "reply" behaviour and the rendering of application/octet-stream parts that are treated as text. David Edmondson (3): emacs: `notmuch-show-insert-part-multipart/encrypted' should not assume the presence of a button. emacs: