[PATCH] bound mail-parse-charset, used by some functions on mm-view.el when there is no charset metadata in the html

[PATCH] bound mail-parse-charset, used by some functions on mm-view.el when there is no charset metadata in the html

[Konrad Scorciapino]

From: Konrad Scorciapino <konrad@scorciapino.com>

---
 emacs/notmuch-show.el |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el
index f3150af..2462ec1 100644
--- a/emacs/notmuch-show.el
+++ b/emacs/notmuch-show.el
@@ -312,9 +312,10 @@ current buffer, if possible."
 (defun notmuch-show-insert-part-*/* (msg part content-type nth depth declared-type)
   ;; This handler _must_ succeed - it is the handler of last resort.
   (notmuch-show-insert-part-header nth content-type declared-type (plist-get part :filename))
-  (let ((content (notmuch-show-get-bodypart-content msg part nth)))
+  (let* ((content (notmuch-show-get-bodypart-content msg part nth))
+         (mail-parse-charset (first (detect-coding-string content))))
     (if content
-	(notmuch-show-mm-display-part-inline msg part content-type content)))
+        (notmuch-show-mm-display-part-inline msg part content-type content)))
   t)
 
 ;; Functions for determining how to handle MIME parts.
-- 
1.7.4.1

Re: [PATCH] bound mail-parse-charset, used by some functions on mm-view.el when there is no charset metadata in the html

[David Edmondson]

[-- Attachment #1: Type: text/plain, Size: 102 bytes --]

Konrad, can you provide a specific example of when this is useful? I
don't understand what it is for.

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [PATCH] bound mail-parse-charset, used by some functions on mm-view.el when there is no charset metadata in the html

[Konrad Scorciapino]

It fixes some encoding issues. I thought it was already incorporated,
as I no longer have this issue, but words like 'català' were being
displayed as 'catala', iirc.

2011/12/30 David Edmondson <dme@dme.org>:
> Konrad, can you provide a specific example of when this is useful? I
> don't understand what it is for.

Re: [PATCH] bound mail-parse-charset, used by some functions on mm-view.el when there is no charset metadata in the html

[David Edmondson]

[-- Attachment #1: Type: text/plain, Size: 458 bytes --]

On Tue, 3 Jan 2012 10:15:44 -0200, Konrad Scorciapino <konr@konr.mobi> wrote:
> It fixes some encoding issues. I thought it was already incorporated,
> as I no longer have this issue, but words like 'català' were being
> displayed as 'catala', iirc.

I'll mark the patch as 'obsolete', then.

> 2011/12/30 David Edmondson <dme@dme.org>:
> > Konrad, can you provide a specific example of when this is useful? I
> > don't understand what it is for.

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]