From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Michael Albinus Newsgroups: gmane.emacs.bugs Subject: bug#53632: Function definition history Date: Fri, 04 Feb 2022 18:00:43 +0100 Message-ID: <87y22qmuro.fsf@gmx.de> References: <87y22spdxo.fsf@gmx.de> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="26791"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) Cc: Glenn Morris , 53632@debbugs.gnu.org To: Stefan Monnier Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Fri Feb 04 18:21:35 2022 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nG2H9-0006jS-9N for geb-bug-gnu-emacs@m.gmane-mx.org; Fri, 04 Feb 2022 18:21:35 +0100 Original-Received: from localhost ([::1]:40036 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1nG2H8-000394-31 for geb-bug-gnu-emacs@m.gmane-mx.org; Fri, 04 Feb 2022 12:21:34 -0500 Original-Received: from eggs.gnu.org ([209.51.188.92]:55186) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1nG1xQ-0007vS-VS for bug-gnu-emacs@gnu.org; Fri, 04 Feb 2022 12:01:13 -0500 Original-Received: from debbugs.gnu.org ([209.51.188.43]:38817) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1nG1xF-000483-V7 for bug-gnu-emacs@gnu.org; Fri, 04 Feb 2022 12:01:12 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1nG1xF-0002vo-R4 for bug-gnu-emacs@gnu.org; Fri, 04 Feb 2022 12:01:01 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Michael Albinus Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Fri, 04 Feb 2022 17:01:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 53632 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 53632-submit@debbugs.gnu.org id=B53632.164399405611250 (code B ref 53632); Fri, 04 Feb 2022 17:01:01 +0000 Original-Received: (at 53632) by debbugs.gnu.org; 4 Feb 2022 17:00:56 +0000 Original-Received: from localhost ([127.0.0.1]:60947 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1nG1xA-0002vN-2R for submit@debbugs.gnu.org; Fri, 04 Feb 2022 12:00:56 -0500 Original-Received: from mout.gmx.net ([212.227.17.20]:50427) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1nG1x7-0002vA-S2 for 53632@debbugs.gnu.org; Fri, 04 Feb 2022 12:00:54 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1643994044; bh=y4xICwx6QKzg5jle/y7a64dv1rDyy8XI7KbdtLqb16Q=; h=X-UI-Sender-Class:From:To:Cc:Subject:References:Date:In-Reply-To; b=iwbdmzdMaXuB5MzFf27Rei7mhkI8oS6DVU3zoLmOMRTiVokXTGxiTqcgv/svxZurC wbgieZvx+NwwVL7WO4Qoo1D6nbU5bCquwPscvp8Jkq2ym8MySZRZMVe9JHdLStpQTs dn9FNkg2X+dnZrtyio9l6/qr1vuZBwC8dXnAmiLc= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Original-Received: from gandalf.gmx.de ([213.220.159.97]) by mail.gmx.net (mrgmx104 [212.227.17.168]) with ESMTPSA (Nemesis) id 1MFbRm-1n05lS2tUk-00H7TH; Fri, 04 Feb 2022 18:00:44 +0100 In-Reply-To: (Stefan Monnier's message of "Fri, 04 Feb 2022 11:30:26 -0500") X-Provags-ID: V03:K1:SAtG0L/R+HbALKa/1RYHF+rKF7JfikDoc5MwaN3dOU68A+JxK6d 0Knc2rO762NI2gLXPIvARTrCeM+GSP/+Hf7lqxMOe3pOrSvFJVy+OYRU2NxMbW8LNmkQZsw +sQvs4FV4ehucnNs3SNPt98RUC+iInu4Qj4/JkZZM6qY3A6oBpIKgsURXbsVqbTfWRxw2dK 4/+fBFC8ga+mpF9DOLG3w== X-UI-Out-Filterresults: notjunk:1;V03:K0:AkMJmi0gQuQ=:NXwWvF14/r7uJ1ypvIUW80 wBFlAxqwWEJaWicCI/ikWYFaaO/gIJ9gI9BQBABT87jZtD8aKsyn4AQXs9fNwELGad0oxE0CP rs3Y4BcFh3fBm05XG/dVD/EkU+zVQZzdRlf2b3gc0Vbza9fgChKEDlDt3JEDuYVxJo5fGk0Eu UkIcEfAoHmFcER7AWTH8NU9nqy5cv4txhKK6AmWR6AqwcthORjRwdOq+m8JJZuYI+T6E0qEv5 xSrb/P1GAxn8tIkWtUvHTbvF8wLKOWgvo2HUYFQAnfitOvIcKCJFu5FQg9CcJH+JgJ8+VXhg4 CFjwHn6HOnI0v5ccCdsqjL6ph7EK8UOdRgLwLktCi5L2Jv9BqXsIa3Ws7ROYPP1+QhdMKxGA2 7pS7cN6Y/sq4tec+ew9duaWsC49+nMD1nefYBkVooVpPE/+7l4T9mmwaPtGUIr0MpfUnjWuow SKBH0lAwSgQOv/rZ1dfuXyMGuk3tInhK5IgZ3fmPs13PGCoFUyZJ20VyGM/2hBQAo0Rivj/GK rn1J18jvCElJiy9tTA1favQTKAvmfveLU6A3wkwPeP8N+ORftDAykuVOHdkbwe6WuvSa7gbTG fhsYVfHuRVcYV/rlA3Bu444bl574W1eP9MneYP1ztafWOhGIRupTY0JCyt4G26swCWVbhQkuP WD7+W/etIHmaVTMuyjcXuoyIjqVYqTNrL368ZKDm/RV/rReLYqTmPg8EDzOLrVDZtLJ5wy9fo Sz3vwFN4mj3a81s+DQ9zLxTdtKLrIEdCryR9fnQxz2b5wvhxRuQsmHNMg7wbOjW5Ci6YDnWz X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:226009 Archived-At: Stefan Monnier writes: Hi Stefan, > Indeed, the behavior is changed, but AFAICT it's "better" in that it > gives us a state closer to the one we had before `tramp.el` was loaded: > the above functions are predefined in `loaddefs.el` > via `;;;###autoload` cookies, so it's normal that they're defined when > `tramp` is not loaded. OK, so be it. > So I think the problem is in the test rather than in the unload code. > The patch below fixes the test for me. The patch essential is to list those functions, and to make an exception. Will work, yes. However, I fear that this is error-prone: Whenever there is another such function with an autoload cookie in the future, the test will fail, again. Perhaps we shall give these functions a symbol property, say `tramp-autoload', and test for that property in order to filter out? This should be more robust. > BTW, I notice that the test uses `functionp` so it doesn't pay attention > to whether macros are properly unloaded (I noticed because I thought it > was odd that the above list didn't include > `tramp-archive-autoload-file-name-regexp` which is similarly predefined > in `loaddefs.el`). Good point, we shall test also for `macrop'. Will extend the test. > Stefan Best regards, Michael.