From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Leo Liu Newsgroups: gmane.emacs.devel Subject: Re: master f995fbd: * lisp/server.el (server-name): Add autoload cookie. (Bug#23576) Date: Fri, 20 May 2016 15:11:45 +0800 Message-ID: References: <20160518194708.27363.47377@vcs.savannah.gnu.org> <20160518194708.86FE0220157@vcs.savannah.gnu.org> <834m9u9dis.fsf@gnu.org> <83r3cx8eg2.fsf@gnu.org> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1463728345 27813 80.91.229.3 (20 May 2016 07:12:25 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Fri, 20 May 2016 07:12:25 +0000 (UTC) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Fri May 20 09:12:16 2016 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1b3ebW-0003dJ-45 for ged-emacs-devel@m.gmane.org; Fri, 20 May 2016 09:12:14 +0200 Original-Received: from localhost ([::1]:53451 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b3ebV-0008LH-BU for ged-emacs-devel@m.gmane.org; Fri, 20 May 2016 03:12:13 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:33934) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b3ebP-0008Kq-Hd for emacs-devel@gnu.org; Fri, 20 May 2016 03:12:08 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b3ebJ-0002kv-G2 for emacs-devel@gnu.org; Fri, 20 May 2016 03:12:06 -0400 Original-Received: from plane.gmane.org ([80.91.229.3]:60693) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b3ebJ-0002kn-8z for emacs-devel@gnu.org; Fri, 20 May 2016 03:12:01 -0400 Original-Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1b3ebD-0003QS-OS for emacs-devel@gnu.org; Fri, 20 May 2016 09:11:56 +0200 Original-Received: from 221.222.145.139 ([221.222.145.139]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Fri, 20 May 2016 09:11:55 +0200 Original-Received: from sdl.web by 221.222.145.139 with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Fri, 20 May 2016 09:11:55 +0200 X-Injected-Via-Gmane: http://gmane.org/ Original-Lines: 26 Original-X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: 221.222.145.139 Face: iVBORw0KGgoAAAANSUhEUgAAACgAAAAoAgMAAADxkFD+AAAADFBMVEUvT09qWs3/pQD///+J kUVcAAAAAWJLR0QAiAUdSAAAAAlwSFlzAAALEwAACxMBAJqcGAAAAAd0SU1FB9cBBwMLOd3veKQA AACuSURBVBjTldE9CgIxEAXgB+lEyFUC2wo5ikdZ8DSypxhMY7H9VuIVwlqkGRgnm59VsHGafIQ3 CZlAtmKIRaHETgYa12lqvEsPYKf8wXHsPGfqPaUM0g9aJPKFXkmNQmSDqwzz4Fpgpz+6WAPY2z5o uPJJpu0uypcl4nyCibMLQ8lCiVjayLoQvw5LsVKQuHPRR958HZbOcVsKeepcLxpByjycGvnKmY+c MBvrtyjfe0vmuLvdq/kAAAAASUVORK5CYII= User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.94 (OS X 10.11.5) Cancel-Lock: sha1:snBT3PYT8SLPa5y9YAp5c/eeHkc= X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 80.91.229.3 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.org gmane.emacs.devel:203898 Archived-At: On 2016-05-20 08:51 +0300, Eli Zaretskii wrote: > Sorry, I don't think I understand what that means. What duplication > do you allude to here? For example if you autoload server-name, the whole (defcustom server-name "server" "The name of the Emacs server, if this Emacs process creates one. The command `server-start' makes use of this. It should not be changed while a server is running." :group 'server :type 'string :version "23.1") is duplicated to the loaddefs file and now you have two definitions in two separate files. > I added the autoload for a reason for which I didn't see a better > solution. If you can suggest a better solution that allows users to > specify the name before starting the first server, please do. Get rid of the set-variable. Nowhere else uses it to demonstrate how to change a defcustom'd variable. Otherwise we have a lot more autoloads to do. Leo