[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?
Thanks,
Štěpán