Re: Removing ignore-errors in mm-dissect-buffer

[Katsumi Yamaoka <yamaoka@jpl.org>]

>>>>> In <v9wtfwo8zf.fsf@marauder.physik.uni-ulm.de> Reiner Steib wrote:

> Gnus errors out with "forward-sexp: Scan error: "Unbalanced
> parentheses", 21, 32" when displaying the following spam article:

I see.  Though the article seems to have been cancelled.

[...]

> The culprit is the string «charset=\"us-ascii\"» in the first MIME
> part which might be invalid:

> ,----
>| --fc55ff24ae7d5dde3b9cddf90cbc0f08
>| Content-Type: text/html; charset=\"us-ascii\"
>| MIME-Version: 1.0
>| Content-Transfer-Encoding: quoted-printable
> `----

> But Gnus should render the article despite of the broken charset
> declaration, I think.

Yes, I think so.  The change I made aimed to cope with
Thunderbird 1.5 which generates wrong name and filename
parameters for an attachment of a long non-ASCII file name[1].
In such a case, now at least type/subtype will be extracted.

> To reproduce (even from «emacs -Q»), eval the following code:

> (require 'mail-parse)
> (with-temp-buffer
>   (insert "Content-Type: text/html; charset=\\\"us-ascii\\\"
> MIME-Version: 1.0
> Content-Transfer-Encoding: quoted-printable")
>   (setq ct (mail-fetch-field "content-type")
> 	ctl (and ct (mail-header-parse-content-type ct))))

> Reverting mm-decode.el to revision 7.37 fixed the problem:

[...]

> If reverting the following changes is not the right thing, could you
> suggest a better approach, Katsumi?

I think mail-header-parse-content-type should never cause erros
and it should extract at least type/subtype if it is possible.
I will make an effort so that it might do so.

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=323318