From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Arthur Miller Newsgroups: gmane.emacs.devel Subject: Re: Patch: perform autoloading when docs is missing from autoload object Date: Sat, 18 Sep 2021 19:31:32 +0200 Message-ID: References: <87ee9o3cqj.fsf@gnus.org> <87o88rxpvy.fsf@gnus.org> <87bl4qt3y3.fsf@gnus.org> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="33074"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) Cc: emacs-devel@gnu.org To: Lars Ingebrigtsen Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Sat Sep 18 19:37:27 2021 Return-path: Envelope-to: ged-emacs-devel@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 1mReHH-0008Mo-5b for ged-emacs-devel@m.gmane-mx.org; Sat, 18 Sep 2021 19:37:27 +0200 Original-Received: from localhost ([::1]:37040 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mReHF-0000A3-Hq for ged-emacs-devel@m.gmane-mx.org; Sat, 18 Sep 2021 13:37:25 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:47022) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mReGX-0007DT-LV for emacs-devel@gnu.org; Sat, 18 Sep 2021 13:36:41 -0400 Original-Received: from mail-he1eur04olkn081a.outbound.protection.outlook.com ([2a01:111:f400:fe0d::81a]:51539 helo=EUR04-HE1-obe.outbound.protection.outlook.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mReGV-00080n-CW for emacs-devel@gnu.org; Sat, 18 Sep 2021 13:36:40 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Wad8H72Qau6tnAQkv1zN5pG7nkKYZ8zJi+XU3j+zAKutYaWZgI8wMYnP7ngGZ2pigyr2kI4Fm83gFsX364ceqzzv0KAGbTTMoZrjQdukDnaL2nWqIZiwmsMFqdmALxNyq6KTu75YELCxeNvdeV658ktPEzG2dsY6OaKq2gtBwoGU31GNEz4b982TwNqjYIS7o8cbOXTgT9AUBkx1MewAnhlFDsQX3WYiFiL7OwxJGa1fQtBj75SwfbTzqfzRHgRqcUWh+AXS+/ktx/6Q3Mnl4uczR55HTuWnPc2cBqzbg5gZhmrutW92jUcj1isQ6XISUKtKqftfgbDunxmeTuu8uA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=IQJ39tQpSGMJ6si3A7vqbvWhAtaK1FLzik+tZEBJmLk=; b=Ej1gUCrE7+YY64IifIJQLj5ECPMcVnHZZqeX0oIPZ5AriopW7f+6/gmKLJ5RtGmt+UPVKRMSv5ByRDE+TfzPDkOXrqV7sxkxWyp+QvbgsRLmYQbwbYAOP3Z2IP26sDIaA8aDQ1BR033dc5vkIqR0vHpQCg1jjTnyOb6tGVX3xmJoBDNfexlrC7tJ63fC2lSR4G/HIjRmGPZ4otZLLnkEGbZt2y6vAf10kfkPYV25YqFY9ik1dWg5ygRO1DT4M/1yNuTNUHq9soppjvppMF9dj1HyXsK2jm5wFmavl8SliRdwC9CR6NOriQNuEjBgr0QwbN1XpGruARjT2HK/87zUtg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=live.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=IQJ39tQpSGMJ6si3A7vqbvWhAtaK1FLzik+tZEBJmLk=; b=gegpBzd7HVAWTN36GG86HYcVpAGkhpoQLLV2viY+9jdOrYbE0nKxxhFnPS6NOq5olUdWB2V6ppzGzf+AaWpmmHadZ1Wiu8bi8mE6KN9GgOdgCFv/i+qdFccAZtuOen0rpl8mSqBcyOdkiO2SNZTnrceTo9fnHNZivR0qvJAQcZ7H2y/2wlOCFgylMG7iYrLeIX4xjEoQceJAVMixajeWZWg25ZCS8zeAT/wrsbvGon9PNzJpQ3iumQdgWvzIQQGIQQBimjOR/hcGhcinBZaDMXc0qslNBOkZkRNqNcKI20BtYDvwnlCBGxw6sC6g3oqWvVH4U3e8Euhh9IEJpfN9cg== Original-Received: from AM9PR09MB4977.eurprd09.prod.outlook.com (2603:10a6:20b:304::20) by AM0PR09MB2594.eurprd09.prod.outlook.com (2603:10a6:208:dd::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4523.17; Sat, 18 Sep 2021 17:31:33 +0000 Original-Received: from AM9PR09MB4977.eurprd09.prod.outlook.com ([fe80::c55c:ece5:bed2:a9dc]) by AM9PR09MB4977.eurprd09.prod.outlook.com ([fe80::c55c:ece5:bed2:a9dc%9]) with mapi id 15.20.4523.018; Sat, 18 Sep 2021 17:31:33 +0000 In-Reply-To: <87bl4qt3y3.fsf@gnus.org> (Lars Ingebrigtsen's message of "Sat, 18 Sep 2021 15:21:08 +0200") X-TMN: [P6StH8eWQluzbTaCKSONgA72mW1ekNvh] X-ClientProxiedBy: AS9PR06CA0007.eurprd06.prod.outlook.com (2603:10a6:20b:462::19) To AM9PR09MB4977.eurprd09.prod.outlook.com (2603:10a6:20b:304::20) X-Microsoft-Original-Message-ID: <874kah4wp7.fsf@live.com> X-MS-Exchange-MessageSentRepresentingType: 1 Original-Received: from pascal.homepc (81.232.177.30) by AS9PR06CA0007.eurprd06.prod.outlook.com (2603:10a6:20b:462::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4523.14 via Frontend Transport; Sat, 18 Sep 2021 17:31:33 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d5b552eb-820d-4b9b-dbef-08d97aca28a9 X-MS-TrafficTypeDiagnostic: AM0PR09MB2594: X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: r+W0fSiWgTdHCxEDGNi8W4PTRFIJMwJWe4tQHyO5dudul66DN5E3ZNFKnggkCjysA4yjHLri/V9YWBEP2b8ai/4lFy2zBdb/CpAtdeeepwgLuKKM9KS8pK46VExwdLBmP/MtRWCBh8Z0UTgrHWJ37p659OyO7lCqg74MWR3z/ql0dEqaGW9s5OU1x1Dlun+v3fEHV/FNigDTKgV94Mg38tWcjH2kTdAovUh7YS/vUg4d4xmTizfZiNQ45At5km5S6A6yxotXTK78MFg8y8ETaI2e2bfKNzvzezusJtD0aAI6yxOq0ndN5vLZ0hdYZmL6/0beS6EOxH4GAETsRkO4xmSsGFRnHBFDtaMvazkeqHbZ+Boq+UTDoqMMFv4vvDR5RzkJFg2vmvaojJHNSzlZUi5zrO/lrm/GCNbfcwk2OVFMckmzbkSI8qM8U9dlQor7 X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: JAggVNQ6gqc7gBTChDSc3YY+DnFl3ac/1Fc50dMYiR2X7HQg3lSe7p4yePhbm7Ct+0v5aa/0bgSCn8bnFfq4UcNw6WdCnvpDSXUe52zhkzhideTRQ9FXpD7EjdK2YdyUb4UUvjaMktjuSgMfTPmSbQ== X-OriginatorOrg: sct-15-20-3174-20-msonline-outlook-72e6e.templateTenant X-MS-Exchange-CrossTenant-Network-Message-Id: d5b552eb-820d-4b9b-dbef-08d97aca28a9 X-MS-Exchange-CrossTenant-AuthSource: AM9PR09MB4977.eurprd09.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Sep 2021 17:31:33.6596 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR09MB2594 Received-SPF: pass client-ip=2a01:111:f400:fe0d::81a; envelope-from=arthur.miller@live.com; helo=EUR04-HE1-obe.outbound.protection.outlook.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, MSGID_FROM_MTA_HEADER=0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 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-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.io gmane.emacs.devel:275020 Archived-At: Lars Ingebrigtsen writes: > Arthur Miller writes: > >> I wanted to later switch to just fetch the docs instead of performing >> load. Loading docs can be done without evaluating the code. It might be a >> better solution anyway? > > If that's possible, that would be preferable, but as you say: > >> By the way, I had thoughts about loading the library vs just docs. I reason, >> that, if I look at some symbol docs via describe-* functions, I am >> probably already using that library/package, or am about to use it, so it >> probably is going to be loaded anyway. At least in many cases, not always of >> course. > > It is quite likely that you want to use the library if you've looked up > the symbol in a *Help* buffer, so why not, indeed, just load it at that > point? If loading the library also helps with some other things -- > highlighting `package-symbol's correctly in the doc string, for > instance. > > So I think I'd be fine with adding the proposed functionality. If there > are people that don't want this, we can add a user option to switch it off. It is already an opt-in option. (defcustom help-enable-symbol-autoload nil "Perform autoload when documentation is not present in autoload object." :type 'boolean :group 'help :version "28.1") There is a defcustom variable that controls this, and it is nil by default, so this is off unless user actively put it on. It would be nice if it is in. Thank you.