all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: David Golden <david.golden@oceanfree.net>
To: emacs-pretest-bug@gnu.org
Subject: Re: 23.0.60; rmail initial misdecoding of iso-8859-15 mail (quoted-printable at least)
Date: Wed, 23 Jan 2008 02:38:10 +0000	[thread overview]
Message-ID: <200801230238.10609.david.golden@oceanfree.net> (raw)
In-Reply-To: <200801190002.20483.david.golden@oceanfree.net>

[apologies for doubling up in that last message; somehow managed
to send an old version of the message body as an additional attachment]

I have made some progress on debugging this, I think: 

rmail-convert-to-babyl-format calls  mail-unquote-printable-region [1] 
with arg "unibyte" set to t - supposed to cause the insertion of the 
decoded characters as unibyte, probably mostly for rmail's 
post-unquoting decoding as it happens (see docstring in the 
mail-unquote-printable-region function).  However, the 
mail-unquote-printable-region function apparently thinks insert-char 
will do this for it [2].  Only, these days, it apparently doesn't, it 
only inserts a unibyte char if buffer is in unibyte mode...

insert-byte exists, so immediate fix would be to change insert-char to 
insert-byte in [2] ?



[1] line 1990 of rmail.el

...
(if quoted-printable-header-field-end
		       (save-excursion
			 (unless
===>			     (mail-unquote-printable-region header-end (point) nil t t)
			   (message "Malformed MIME quoted-printable message")
...


[2] line 147 of mail-utils.el
...
		     (if unibyte
			 (progn
			   (replace-match "")
===>			   ;; insert-char will insert this as unibyte,
			   (insert-char char 1))
		       (replace-match (make-string 1 char) t t))))
....

      reply	other threads:[~2008-01-23  2:38 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-19  0:02 23.0.60; rmail initial misdecoding of iso-8859-15 mail (quoted-printable at least) David Golden
2008-01-23  2:38 ` David Golden [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200801230238.10609.david.golden@oceanfree.net \
    --to=david.golden@oceanfree.net \
    --cc=emacs-pretest-bug@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.