GNU bug report logs - #73386
[PATCH] Fix keybinding support for *-auto-reveal

Previous Next

Full log

Message #17 received at 73386 <at> debbugs.gnu.org (full text, mbox):

From: Paul Nelson <ultrono <at> gmail.com>
To: Arash Esbati <arash <at> gnu.org>
Cc: 73386 <at> debbugs.gnu.org
Subject: Re: bug#73386: [PATCH] Fix keybinding support for *-auto-reveal
Date: Thu, 26 Sep 2024 20:43:04 +0200

[Message part 1 (text/plain, inline)]

Seems fine on my end, so I've attached the updated patch.

Thanks, best,
Paul

On Wed, Sep 25, 2024 at 10:05 AM Arash Esbati <arash <at> gnu.org> wrote:
>
> Paul Nelson <ultrono <at> gmail.com> writes:
>
> >> Thanks.  I have one question: Is it possible to set the LEXICAL argument
> >> of `eval' to t?
> >
> > If I understand correctly, your suggestion is to use something like
> > the following?
>
> Yes, plus the one addition below.
>
> > (defcustom preview-auto-reveal
> >       '(eval . ((apply #'preview-arrived-via
> >                        (mapcar (lambda (cmd)
> >                                  (if (and (listp cmd) (eq (car cmd) 'key-binding))
> >                                      (eval cmd)
>                                        (eval cmd t)
> >                                    cmd))
> >                                preview-auto-reveal-commands))
> >                 t))
> > ...)
> >
> > I assume this will have no effect, but would be happy to test it out
> > for a couple days.
>
> Great, thanks, looking forward to your feedback.
>
> Best, Arash

[0001-Fix-keybinding-support-for-auto-reveal.patch (application/octet-stream, attachment)]

Previous Next

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