GNU bug report logs - #60740
29.0.60; Broken isearch-emoji-by-name

Previous Next

Full log

View this message in rfc822 format

From: Jonas Bernoulli <jonas <at> bernoul.li>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 60740 <at> debbugs.gnu.org, larsi <at> gnus.org
Subject: bug#60740: [PATCH 0/2] emoji changes
Date: Sun, 05 Feb 2023 17:29:12 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Jonas Bernoulli <jonas <at> bernoul.li>
>> Cc: 60740 <at> debbugs.gnu.org, larsi <at> gnus.org
>> Date: Sun, 05 Feb 2023 15:40:48 +0100
>> 
>> Eli Zaretskii <eliz <at> gnu.org> writes:
>> 
>> > My suggestion is for now to install the first patch on emacs-29 and
>> > the second one on master.
>> 
>> I've installed the first
>
> This apparently leads to

I looked at the log to prevent that, but I think I just redirected
stdout and forgot about stderr.  Byte-compiler warnings go to stderr,
right?

>     ELC      ../lisp/isearch.elc
>
>   In end of data:
>   isearch.el:2785:40: Warning: the function `emoji--read-emoji' is not
>   known to be defined.

What's the usual way of dealing with this in Emacs itself?  I wasn't
sure using an autoload would work here, and was going to use another
approach if this resulted in a warning.  Unfortunately I missed the
warning.  Should I use 'declare-function' instead and require 'emoji'
inside 'isearch-emoji-by-name'?

>     ELC      international/emoji.elc
>
>   In emoji--choose-emoji:
>   international/emoji.el:688:37: Warning: Unused lexical variable
>   `emoji--read-emoji'

Yikes.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.