unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* mm-with-unibyte-current-buffer broken in Emacs trunk
@ 2010-05-11 10:25 Sascha Wilde
  2010-05-11 23:28 ` Katsumi Yamaoka
  0 siblings, 1 reply; 2+ messages in thread
From: Sascha Wilde @ 2010-05-11 10:25 UTC (permalink / raw)
  To: emacs-devel; +Cc: ding

Hi *,

with an current build of GNU Emacs from bzr I can not forward
(gnus-summary-mail-forward) mail any more.

The problem boils down to message-forward-make-body-mml calling
mm-with-unibyte-current-buffer which in turn tries to set the read only
variable enable-multibyte-characters.

According to this comment in mm-with-unibyte-current-buffer:

  ;; FIXME: (default-value 'enable-multibyte-characters) is read-only
  ;; so let-binding it is wrong.  The right fix is to not use this
  ;; macro at all any more, since it's been ill-defined from the start.

the problem is known, but I'd say not being able to forward messages
(and there might be other serious problems, too) is an regression, so
this _really_ needs fixing.

FWIW: simply removing the call to mm-with-unibyte-current-buffer from
message-forward-make-body-mml worked for me -- but I suspect there are
corner cases where this would cause problems.

cheers
sascha
-- 
Sascha Wilde : xedit? Das sieht zwar wie vi aus als könne es nix, aber im
             : Gegensatz zu vi kann es wirklich nix und nix ist noch geschönt!
             : (Michael Core in dafc)



^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: mm-with-unibyte-current-buffer broken in Emacs trunk
  2010-05-11 10:25 mm-with-unibyte-current-buffer broken in Emacs trunk Sascha Wilde
@ 2010-05-11 23:28 ` Katsumi Yamaoka
  0 siblings, 0 replies; 2+ messages in thread
From: Katsumi Yamaoka @ 2010-05-11 23:28 UTC (permalink / raw)
  To: Sascha Wilde; +Cc: ding, emacs-devel

>>>>> Sascha Wilde wrote:
> with an current build of GNU Emacs from bzr I can not forward
> (gnus-summary-mail-forward) mail any more.

> The problem boils down to message-forward-make-body-mml calling
> mm-with-unibyte-current-buffer which in turn tries to set the read only
> variable enable-multibyte-characters.

[...]

The day before yesterday I just noticed the message forwarding
doesn't work, and fixed the code temporarily in Git and Bzr repos.

2010-05-10  Katsumi Yamaoka  <yamaoka@jpl.org>

	* message.el (message-forward-make-body-plain)
	(message-forward-make-body-mml):
	Don't use mm-with-unibyte-current-buffer.

But I have a plan to fix it further; that is to make the forwarding
codes assume that an original message to be forwarded is always
a multibyte string.  Actually an original message always comes from
only the " *Original Article*" buffer in which the multibyteness
is turned on, AFAIK.



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-05-11 23:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-11 10:25 mm-with-unibyte-current-buffer broken in Emacs trunk Sascha Wilde
2010-05-11 23:28 ` Katsumi Yamaoka

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).