From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>
To: David Edmondson <dme@dme.org>, notmuch@notmuchmail.org
Subject: Re: [PATCH] notmuch/emacs: Observe the charset of encoded parts,
	where known.
In-Reply-To: <1326279001-28427-1-git-send-email-dme@dme.org>
References: <1326279001-28427-1-git-send-email-dme@dme.org>
User-Agent: Notmuch/0.10.2+135~gb811a3c (http://notmuchmail.org) Emacs/23.3.1
	(x86_64-pc-linux-gnu)
Date: Wed, 11 Jan 2012 22:34:45 +0400
Message-ID: <87k44ym6ka.fsf@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Precedence: list

-1

I already tried to solve the above problem using a more general approach
(output all content-type parameters, not just charset) [1].  There was a
lengthy discussion on IRC about it and it was rejected.  The consensus
was that we need to make some more substantial changes to JSON and raw
output formats to properly fix the issue (and the issue is more general
than what this patch fixes, other content-type parameters are useful for
renderers as well and should be included in the output).

One particular issue with your patch is that it adds (incorrect) charset
parameter to plain/text parts which are converted to UTF-8 in JSON
output.

I am planning to work on a proper fix for this issue, but decided to
postpone it until Austin's rewrite of notmuch show is complete.

Regards,
  Dmitry

[1] id:"1321659905-24367-1-git-send-email-dmitry.kurochkin@gmail.com"