From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Luc Teirlinck Newsgroups: gmane.emacs.devel Subject: Re: custom-declare-variable Date: Tue, 5 Jul 2005 18:55:37 -0500 (CDT) Message-ID: <200507052355.j65NtbZ21516@raven.dms.auburn.edu> References: <200507042320.j64NKNe15629@raven.dms.auburn.edu> <200507042337.j64Nbel15654@raven.dms.auburn.edu> NNTP-Posting-Host: main.gmane.org X-Trace: sea.gmane.org 1120610472 2908 80.91.229.2 (6 Jul 2005 00:41:12 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Wed, 6 Jul 2005 00:41:12 +0000 (UTC) Cc: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Jul 06 02:41:07 2005 Return-path: Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1Dpxy7-0007s7-Q7 for ged-emacs-devel@m.gmane.org; Wed, 06 Jul 2005 02:40:52 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DpxzP-0002IK-CP for ged-emacs-devel@m.gmane.org; Tue, 05 Jul 2005 20:42:11 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1DpxxS-0000wd-Ul for emacs-devel@gnu.org; Tue, 05 Jul 2005 20:40:11 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1DpxxQ-0000vC-Iz for emacs-devel@gnu.org; Tue, 05 Jul 2005 20:40:10 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DpxuF-0007cr-AC for emacs-devel@gnu.org; Tue, 05 Jul 2005 20:36:51 -0400 Original-Received: from [131.204.53.104] (helo=manatee.dms.auburn.edu) by monty-python.gnu.org with esmtp (Exim 4.34) id 1DpxOe-0003UW-E3 for emacs-devel@gnu.org; Tue, 05 Jul 2005 20:04:12 -0400 Original-Received: from raven.dms.auburn.edu (raven.dms.auburn.edu [131.204.53.29]) by manatee.dms.auburn.edu (8.12.10/8.12.10) with ESMTP id j65NvfCK004013; Tue, 5 Jul 2005 18:57:41 -0500 (CDT) Original-Received: (from teirllm@localhost) by raven.dms.auburn.edu (8.11.7p1+Sun/8.11.7) id j65NtbZ21516; Tue, 5 Jul 2005 18:55:37 -0500 (CDT) X-Authentication-Warning: raven.dms.auburn.edu: teirllm set sender to teirllm@dms.auburn.edu using -f Original-To: abraham@dina.kvl.dk In-reply-to: (message from Per Abrahamsen on Tue, 05 Jul 2005 11:37:52 +0200) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:40480 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:40480 Per Abrahamsen wrote: Would it be possible to undo the patch that integrated custom themes into Emacs cvs? It might be easier to redo theme support from scratch, than make the current code work... I mean, if nobody really understand the ideas behind the design of custom themes, it is unlikely that stepwise refinement of the code will produce something useful. cus-theme.el is probably usable, even with a completely rewritten Themes code in custom.el, although it would need bug fixes and improvements. In as far as the Themes code in custom.el is concerned, I do not even understand what it is _trying_ to do and some things about it make me very wary. There is the strangeness in `custom-declare-variable' which I pointed out. There is `custom-do-theme-reset' whose docstring describes _twice_ what the function does, giving two completely contradictory descriptions. I read the code carefully to see which of the two was correct. The actual behavior has nothing to do with either description. Things like that make you wonder. Maybe somebody else can take a look at the Themes code in custom.el and see whether they understand any of it. Otherwise, I believe that, at best, some parts of it maybe could be recycled into the new design we would come up with, but still it means essentially starting over from scratch. Sincerely, Luc.