mm-codepage-setup not quite right

mm-codepage-setup not quite right

[Kevin Ryde]

In debian emacs 21.4 and the current gnus cvs, the call

	(mm-charset-to-coding-system 'windows-1251)

returns nil, where I think the recent code is supposed to make it do a
codepage setup.

I think the problem is that the call

	(mm-codepage-setup 1251 t)

doesn't add windows-1251 to mm-charset-synonym-alist, the way the rest
of the code expects.  The test in mm-codepage-setup

	(mm-coding-system-p alias)

looks dubious.  I don't think the alias, in this case symbol
windows-1251, will be known to coding-system-p.


I noticed this when thinking to add some similar stuff myself, but
cvs-up showed great minds think alike.  I don't get any real mail in
windows-1251, only spam, but seeing cyrillic spam correctly is
important, isn't it?  :-)

Re: mm-codepage-setup not quite right

[Reiner Steib]

On Thu, Oct 27 2005, Kevin Ryde wrote:

> I think the problem is that the call
>
> 	(mm-codepage-setup 1251 t)
>
> doesn't add windows-1251 to mm-charset-synonym-alist, the way the rest
> of the code expects.  The test in mm-codepage-setup
>
> 	(mm-coding-system-p alias)
>
> looks dubious.  I don't think the alias, in this case symbol
> windows-1251, will be known to coding-system-p.

Indeed.  I don't remember why I added this test.  I'll remove it.

> I noticed this when thinking to add some similar stuff myself, but
> cvs-up showed great minds think alike.

:-)

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/