From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.io/gmane.emacs.gnus.general/64569 Path: news.gmane.org!not-for-mail From: Katsumi Yamaoka Newsgroups: gmane.emacs.gnus.general Subject: Re: customizing mm-charset-synonym-alist Date: Fri, 27 Apr 2007 07:42:48 +0900 Organization: Emacsen advocacy group Message-ID: References: <86abwwtq52.fsf@ra333.heimat.gr.jp> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1177627424 4936 80.91.229.12 (26 Apr 2007 22:43:44 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Thu, 26 Apr 2007 22:43:44 +0000 (UTC) To: ding@gnus.org Original-X-From: ding-owner+M13080@lists.math.uh.edu Fri Apr 27 00:43:41 2007 Return-path: Envelope-to: ding-account@gmane.org Original-Received: from util0.math.uh.edu ([129.7.128.18]) by lo.gmane.org with esmtp (Exim 4.50) id 1HhCgf-0003UX-ER for ding-account@gmane.org; Fri, 27 Apr 2007 00:43:41 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.math.uh.edu) by util0.math.uh.edu with smtp (Exim 4.63) (envelope-from ) id 1HhCg4-0008A3-Kh; Thu, 26 Apr 2007 17:43:04 -0500 Original-Received: from mx1.math.uh.edu ([129.7.128.32]) by util0.math.uh.edu with esmtps (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1HhCg3-00089o-B1 for ding@lists.math.uh.edu; Thu, 26 Apr 2007 17:43:03 -0500 Original-Received: from quimby.gnus.org ([80.91.231.51]) by mx1.math.uh.edu with esmtp (Exim 4.63) (envelope-from ) id 1HhCg1-0000Xi-Mk for ding@lists.math.uh.edu; Thu, 26 Apr 2007 17:43:03 -0500 Original-Received: from orlando.hostforweb.net ([216.246.45.90]) by quimby.gnus.org with esmtp (Exim 3.35 #1 (Debian)) id 1HhCg0-0003be-00 for ; Fri, 27 Apr 2007 00:43:00 +0200 Original-Received: from [66.225.201.151] (port=35172 helo=mail.jpl.org) by orlando.hostforweb.net with esmtpa (Exim 4.63) (envelope-from ) id 1HhCg2-0001BH-UW for ding@gnus.org; Thu, 26 Apr 2007 17:43:03 -0500 X-Hashcash: 1:20:070426:ding@gnus.org::uinVkJwN4Vm3eh0E:0000DrtE X-Face: #kKnN,xUnmKia.'[pp`;Omh}odZK)?7wQSl"4o04=EixTF+V[""w~iNbM9ZL+.b*_CxUmFk B#Fu[*?MZZH@IkN:!"\w%I_zt>[$nm7nQosZ<3eu;B:$Q_:p!',P.c0-_Cy[dz4oIpw0ESA^D*1Lw= L&i*6&( User-Agent: Gnus/5.110006 (No Gnus v0.6) Emacs/22.1.50 (gnu/linux) Cancel-Lock: sha1:k8ERqsUxGaJeu3FUmh7/AxSi+Mk= X-Antivirus-Scanner: Clean mail though you should still use an Antivirus X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - orlando.hostforweb.net X-AntiAbuse: Original Domain - gnus.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - jpl.org X-Source: X-Source-Args: X-Source-Dir: X-Spam-Score: -2.5 (--) List-ID: Precedence: bulk Xref: news.gmane.org gmane.emacs.gnus.general:64569 Archived-At: >>>>> In >>>>> Reiner Steib wrote: > On Thu, Apr 26 2007, Katsumi Yamaoka wrote: >> I've also made it use defcustom, so you can now customize the alist >> easily. > I think this is not a good idea. > 1. `mm-charset-synonym-alist' is modified by mm-setup-codepage-*, so a > user will see "CHANGED outside Customize; operating on it here may be > unreliable." Oops. I fully agree to not making it a user option. > ,---- >| ;; Initialize: >| (mm-setup-codepage-iso-8859) >| (mm-setup-codepage-ibm) > `---- > 2. The initial value depends on the availability of charsets (lots of > `mm-coding-system-p's). So it will change when using different > Emacs versions and flavors. But if the user has customized it, > this conditionals are eliminated. > IMHO, if we want a customizable variable, we should create a new > variable (default: nil) and construct `mm-charset-synonym-alist' from > the new one, `mm-codepage-iso-8859-list' and `mm-codepage-ibm-list' > (using mm-setup-codepage-*). A new setup function should check if the > cdr of each element is a valid charset and add it to > `mm-charset-synonym-alist'. I see. Otherwise, it might not be necessary that a variable has ones like `(iso8859-1 . iso-8859-1)' since such ones are able to be corrected easily by a hard-wired program. > BTW, if former defvars are changed to defcustoms, the :version is the > one when it's made customizable, not when the defvar was introduced. > So ":version "21.1" is not correct. I must keep it in mind. Thank you. Anyway, I've made `mm-charset-synonym-alist' revert to using `defvar'. Regards,