[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#44651: [PATCH] Silence byte-compiler warning
From: |
Brian Leung |
Subject: |
bug#44651: [PATCH] Silence byte-compiler warning |
Date: |
Tue, 17 Nov 2020 00:52:41 +0100 (CET) |
> I don't think that should be necessary? The function in question is
> autoloaded:
>
> ;;;###autoload
>
> Where/when do you see this compilation warning?
I noticed this on the feature/native-comp branch, after invoking
`helm-man-woman` and having woman.el native-compiled asynchronously. I don't
have the log.
I did not notice that it was autoloaded. Andrea, are you aware of any issues
that the native-comp branch may have with this kind of thing (autoloaded
functions defined in files that do not necessarily get REQUIRE-d)? If not,
maybe we can just ignore my patch.
Thanks,
Brian
> ----------------------------------------
> From: Lars Ingebrigtsen <larsi@gnus.org>
> Sent: Mon Nov 16 23:46:05 CET 2020
> To: Brian Leung <leungbk@mailfence.com>
> Cc: <44651@debbugs.gnu.org>
> Subject: Re: bug#44651: [PATCH] Silence byte-compiler warning
>
>
> Brian Leung <leungbk@mailfence.com> writes:
>
> > See attached.
>
> [...]
>
> > * lisp/woman.el: Declare imenu-add-to-menubar.
>
> I don't think that should be necessary? The function in question is
> autoloaded:
>
> ;;;###autoload
> (defun imenu-add-to-menubar (name)
>
> Where/when do you see this compilation warning?
>
> --
> (domestic pets only, the antidote for overdose, milk.)
> bloggy blog: http://lars.ingebrigtsen.no
--
Sent with https://mailfence.com
Secure and private email