[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#39842: [PATCH] Use help-fns-short-filename in other describe- comman

From: Štěpán Němec
Subject: bug#39842: [PATCH] Use help-fns-short-filename in other describe- commands
Date: Sat, 29 Feb 2020 21:18:33 +0100
User-agent: Notmuch/0.29.3 (https://notmuchmail.org) Emacs/28.0.50 (x86_64-pc-linux-gnu)

On Sat, 29 Feb 2020 15:02:05 -0500
Stefan Monnier wrote:

>> +;;;###autoload
>>  (defun help-fns-short-filename (filename)
> OK, we can live that.  Maybe, moving it to help.el would be a cleaner option.
>> index 45cbaad4e8..32204d04e3 100644
>> --- a/lisp/help.el
>> +++ b/lisp/help.el
>> @@ -879,6 +879,7 @@ describe-key
>>              (princ ", which is ")
>>          (describe-function-1 defn)))))))
>> +(declare-function help-fns-short-filename "help-fns" (filename))
>>  (defun describe-mode (&optional buffer)
>>    "Display documentation of current major mode and minor modes.
>>  A brief summary of the minor modes comes first, followed by the
> The autoload cookie above should render this `declare-function`
> unnecessary, right?

R...right...? I'm somewhat unclear on the "something non-preloaded
required by something preloaded" situation. :-)

So, which shall it be? Move help-fns-short-filename to help.el, or
just remove the declaration?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]