unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Additional autoload-coding-system entries in code-pages.el
@ 2005-03-01 13:50 Reiner Steib
  2005-03-01 16:07 ` Stefan Monnier
  2005-03-03 11:04 ` Additional autoload-coding-system entries in code-pages.el Richard Stallman
  0 siblings, 2 replies; 6+ messages in thread
From: Reiner Steib @ 2005-03-01 13:50 UTC (permalink / raw)


[-- Attachment #1: Type: text/plain, Size: 768 bytes --]

Hi,

`code-pages.el' provides various coding systems.  But only defines
autoloads for pt154 and iso-8859-11:

,----
| 2 matches for "###autoload(autoload-coding-system" in buffer: code-pages.el.~1.23~
|    4462:;;;###autoload(autoload-coding-system 'pt154 '(require 'code-pages))
|    4472:;;;###autoload(autoload-coding-system 'iso-8859-11 '(require 'code-pages))
`----

I propose to add autoloads for all iso-8859-* and windows-125* coding
systems.  With these autoload, Gnus (and probably also other Emacs
based mail and news readers) are able to display articles with the
corresponding MIME charsets correctly.

A patch is attached.  Okay to commit?

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

[-- Attachment #2: code-pages-autoload.patch --]
[-- Type: text/x-patch, Size: 2674 bytes --]

--- code-pages.el	14 Jun 2004 02:35:11 +0200	1.23
+++ code-pages.el	01 Mar 2005 14:15:46 +0100	
@@ -2440,6 +2440,7 @@
   nil
   nil])
 
+;;;###autoload(autoload-coding-system 'windows-1250 '(require 'code-pages))
 (cp-make-coding-system
  windows-1250
  [?\€
@@ -2571,6 +2572,7 @@
   ?\ţ
   ?\˙])
 
+;;;###autoload(autoload-coding-system 'windows-1253 '(require 'code-pages))
 (cp-make-coding-system
  windows-1253
  [?\€
@@ -2703,6 +2705,7 @@
   nil]
  nil ?g) ;; Greek
 
+;;;###autoload(autoload-coding-system 'windows-1254 '(require 'code-pages))
 (cp-make-coding-system
  windows-1254
  [?\€
@@ -2835,6 +2838,7 @@
   ?\ÿ])
 
 ;; yi_US
+;;;###autoload(autoload-coding-system 'windows-1255 '(require 'code-pages))
 (cp-make-coding-system
  windows-1255
  [?\€
@@ -2967,6 +2971,7 @@
   nil]
  nil ?h) ;; Hebrew
 
+;;;###autoload(autoload-coding-system 'windows-1256 '(require 'code-pages))
 (cp-make-coding-system
  windows-1256
  [?\€
@@ -3099,6 +3104,7 @@
   ?\ے]
  nil ?a) ;; Arabic
 
+;;;###autoload(autoload-coding-system 'windows-1257 '(require 'code-pages))
 (cp-make-coding-system
  windows-1257
  [?\€
@@ -3230,6 +3236,7 @@
   ?\ž
   nil])
 
+;;;###autoload(autoload-coding-system 'windows-1258 '(require 'code-pages))
 (cp-make-coding-system
  windows-1258
  [?\€
@@ -3652,6 +3659,7 @@
 ;;   ISO 8859-16 was primarily designed for single-byte encoding the Romanian
 ;;   language. The UTF-8 charset is the preferred and in today's MIME software
 ;;   more widely implemented encoding suitable for Romanian.
+;;;###autoload(autoload-coding-system 'iso-8859-16 '(require 'code-pages))
 (cp-make-coding-system
  iso-latin-10				; consistent with, e.g. Latin-1
  [nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
@@ -3823,6 +3831,7 @@
  ?6)
 (define-coding-system-alias 'arabic-iso-8bit 'iso-8859-6)
 
+;;;###autoload(autoload-coding-system 'iso-8859-10 '(require 'code-pages))
 (cp-make-coding-system
  iso-latin-6
  [nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
@@ -3929,6 +3938,7 @@
 (define-coding-system-alias 'latin-6 'iso-latin-6)
 
 ;; used by lt_LT, lv_LV, mi_NZ
+;;;###autoload(autoload-coding-system 'iso-8859-13 '(require 'code-pages))
 (cp-make-coding-system
  iso-latin-7
  [nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
@@ -4039,6 +4049,7 @@
 ;; Fixme: check on the C1 characters which libiconv includes.  They
 ;; are reproduced below, but are probably wrong.  I can't find an
 ;; official definition of georgian-ps.
+;;;###autoload(autoload-coding-system 'georgian-ps '(require 'code-pages))
 (cp-make-coding-system
  georgian-ps				; used by glibc for ka_GE
  [?\€

[-- Attachment #3: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

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

* Re: Additional autoload-coding-system entries in code-pages.el
  2005-03-01 13:50 Additional autoload-coding-system entries in code-pages.el Reiner Steib
@ 2005-03-01 16:07 ` Stefan Monnier
  2005-03-04 13:51   ` Reiner Steib
  2005-03-03 11:04 ` Additional autoload-coding-system entries in code-pages.el Richard Stallman
  1 sibling, 1 reply; 6+ messages in thread
From: Stefan Monnier @ 2005-03-01 16:07 UTC (permalink / raw)


> `code-pages.el' provides various coding systems.  But only defines
> autoloads for pt154 and iso-8859-11:

> ,----
> | 2 matches for "###autoload(autoload-coding-system" in buffer: code-pages.el.~1.23~
> |    4462:;;;###autoload(autoload-coding-system 'pt154 '(require 'code-pages))
> |    4472:;;;###autoload(autoload-coding-system 'iso-8859-11 '(require 'code-pages))
> `----

> I propose to add autoloads for all iso-8859-* and windows-125* coding
> systems.  With these autoload, Gnus (and probably also other Emacs
> based mail and news readers) are able to display articles with the
> corresponding MIME charsets correctly.

I every once in a while receive email (or see newsposts) with windows-1256
encoding (i.e. an arabic encoding) because of brain-dead software which
selects windows-1256 instead of some latin-1 variant (typically those
messages contain text with a few é, which happen to also exist in
window-1256).

So those autoloads would indeed be very helpful,


        Stefan

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

* Re: Additional autoload-coding-system entries in code-pages.el
  2005-03-01 13:50 Additional autoload-coding-system entries in code-pages.el Reiner Steib
  2005-03-01 16:07 ` Stefan Monnier
@ 2005-03-03 11:04 ` Richard Stallman
  1 sibling, 0 replies; 6+ messages in thread
From: Richard Stallman @ 2005-03-03 11:04 UTC (permalink / raw)
  Cc: emacs-devel

It seems good it me.  Handa, do you agree?

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

* Re: Additional autoload-coding-system entries in code-pages.el
  2005-03-01 16:07 ` Stefan Monnier
@ 2005-03-04 13:51   ` Reiner Steib
  2005-03-07 15:18     ` Recursive `require' for feature `code-pages' (was: Additional autoload-coding-system entries in code-pages.el) Reiner Steib
  0 siblings, 1 reply; 6+ messages in thread
From: Reiner Steib @ 2005-03-04 13:51 UTC (permalink / raw)
  Cc: Richard Stallman, Stefan Monnier

On Tue, Mar 01 2005, Stefan Monnier wrote:

> Reiner Steib wrote:
>> I propose to add autoloads for all iso-8859-* and windows-125* coding
>> systems.  With these autoload, Gnus (and probably also other Emacs
>> based mail and news readers) are able to display articles with the
>> corresponding MIME charsets correctly.
[...]
> So those autoloads would indeed be very helpful,

On Thu, Mar 03 2005, Richard Stallman wrote:

> It seems good it me.  Handa, do you agree?

As nobody seems to object, I have installed the patch.

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

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

* Recursive `require' for feature `code-pages' (was: Additional autoload-coding-system entries in code-pages.el)
  2005-03-04 13:51   ` Reiner Steib
@ 2005-03-07 15:18     ` Reiner Steib
  2005-03-11 12:01       ` Kenichi Handa
  0 siblings, 1 reply; 6+ messages in thread
From: Reiner Steib @ 2005-03-07 15:18 UTC (permalink / raw)


[-- Attachment #1: Type: text/plain, Size: 1431 bytes --]

On Fri, Mar 04 2005, Reiner Steib wrote:

> On Tue, Mar 01 2005, Stefan Monnier wrote:
>
>> Reiner Steib wrote:
>>> I propose to add autoloads for all iso-8859-* and windows-125* coding
>>> systems.  With these autoload, Gnus (and probably also other Emacs
>>> based mail and news readers) are able to display articles with the
>>> corresponding MIME charsets correctly.
> [...]
>> So those autoloads would indeed be very helpful,
>
> On Thu, Mar 03 2005, Richard Stallman wrote:
>
>> It seems good it me.  Handa, do you agree?
>
> As nobody seems to object, I have installed the patch.

There's a problem with this:

- Save the attached file as "Latin-7.txt".

- Start Emacs:
  $ LC_ALL=C emacs -Q Latin-7.txt

- C-h v buffer-file-coding-system RET gives:

,----
| buffer-file-coding-system's value is iso-latin-7
| Local in buffer Latin-7.txt; global value is nil
| Automatically becomes buffer-local when set in any fashion.
`----

But the non-ASCII characters are displayed as \377, and the mode line
doesn't display latin-7 but undecided.

- In the *Messages* buffer, I get:

   Error during redisplay:
   (error Recursive `require' for feature `code-pages') [9 times]

  `M-: (featurep 'code-pages) RET' gives t.

- When loading the file "Latin-7.txt" again, the non-ASCII characters
  are displayed correctly.

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

[-- Attachment #2: Latin-7 aka iso-8859-13 sample --]
[-- Type: text/plain, Size: 231 bytes --]

;; Text in Latin-7 aka iso-8859-13 coding:
(require ’pc-mode) ;; Wrong: ’ instead of '
(pc-selection-mode)
(require ’tex-site) ;; Wrong: ’ instead of '
Grüße

;; Local Variables:
;; coding: iso-8859-13
;; End:


[-- Attachment #3: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

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

* Re: Recursive `require' for feature `code-pages' (was: Additional autoload-coding-system entries in code-pages.el)
  2005-03-07 15:18     ` Recursive `require' for feature `code-pages' (was: Additional autoload-coding-system entries in code-pages.el) Reiner Steib
@ 2005-03-11 12:01       ` Kenichi Handa
  0 siblings, 0 replies; 6+ messages in thread
From: Kenichi Handa @ 2005-03-11 12:01 UTC (permalink / raw)
  Cc: emacs-devel

In article <v97jkjjxhv.fsf@marauder.physik.uni-ulm.de>, Reiner Steib <reinersteib+gmane@imap.cc> writes:

> There's a problem with this:

> - Save the attached file as "Latin-7.txt".

> - Start Emacs:
>   $ LC_ALL=C emacs -Q Latin-7.txt

> - C-h v buffer-file-coding-system RET gives:

> ,----
> | buffer-file-coding-system's value is iso-latin-7
> | Local in buffer Latin-7.txt; global value is nil
> | Automatically becomes buffer-local when set in any fashion.
> `----

> But the non-ASCII characters are displayed as \377, and the mode line
> doesn't display latin-7 but undecided.

I've just installed a fix.

---
Ken'ichi HANDA
handa@m17n.org

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

end of thread, other threads:[~2005-03-11 12:01 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-03-01 13:50 Additional autoload-coding-system entries in code-pages.el Reiner Steib
2005-03-01 16:07 ` Stefan Monnier
2005-03-04 13:51   ` Reiner Steib
2005-03-07 15:18     ` Recursive `require' for feature `code-pages' (was: Additional autoload-coding-system entries in code-pages.el) Reiner Steib
2005-03-11 12:01       ` Kenichi Handa
2005-03-03 11:04 ` Additional autoload-coding-system entries in code-pages.el Richard Stallman

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).