Re: eldoc errors in elisp

[T.V Raman]

Awesome, will rewrite once I hear back.

One constraint -- I'd like to keep backwrd compatibility with Emacs 28
since Emacs 29 is not yet ready for release.

I myself run Emacs from Git@Head



João Távora writes:
 > Alright, i had a look and it should be easy to fix.
 > 
 > I'll do that later.
 > 
 > I have a feeling we already had this conversation about an year ago. Your
 > code is fine for the most part, but you must rewrite it to use the
 > arguments passed to it instead of consulting internal implementation
 > details.
 > 
 > João
 > 
 > On Sat, Apr 22, 2023, 19:13 T.V Raman <raman@google.com> wrote:
 > 
 > >
 > > (Copying emacspeak list so it gets archived and found later)
 > >
 > > Thanks for the offer to help, I'll take you up on it, see below.
 > >
 > > Your guess is correct.
 > >
 > > There are two goals for Emacspeak visavis Eldoc
 > >
 > > 1. If eldoc auto triggers a message, and if emacspeak were to speak it
 > >    in every case, it gets annoying fast and you cant do what you're
 > >    doing.
 > >
 > > The advantage with visually showing the message is that the eye can
 > >    choose to ignore it; the ear cannot ignore something
 > >    spoken. Emacspeak tries to somewhat ameliorate this by speaking
 > >    notifications just on one ear, but it's still too much in the case
 > >    of eldoc.
 > >
 > >    2. Complementary requirement; you want to hear that message because
 > >       it does have helpful info; so you want it spoken "on demand".
 > >
 > >       Solution:
 > >
 > >    Emacspeak produces a very short auditory icon to
 > >       indicate that eldoc is available at point as you move through
 > >       the code; the experienced user can then hit a key and here it.
 > >
 > >       Here is the implementation; it's pretty short, and you might be
 > >       able to advice me (no defadvice pun  intended)
 > >       See function emacspeak-speak-eldoc
 > >       at
 > > https://github.com/tvraman/emacspeak/blob/master/lisp/emacspeak-advice.el#L801
 > >
 > > --
 > >
 > > Thanks,
 > >
 > > --Raman(I Search, I Find, I Misplace, I Research)
 > > ♉ Id: kg:/m/0285kf1  🦮
 > >
 > > --
 > >
 > > Thanks,
 > >
 > > --Raman(I Search, I Find, I Misplace, I Research)
 > > ♉ Id: kg:/m/0285kf1  🦮
 > >

-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮

--

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮