unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#52374: keymap-lookup with prefixes returns nil
@ 2021-12-08 12:21 Stefan Kangas
  2021-12-10 12:11 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Kangas @ 2021-12-08 12:21 UTC (permalink / raw)
  To: 52374

Run emacs -Q, type `M-x re-builder RET' and then `C-c C-q':

Now evaluate:

    (lookup-key reb-mode-map "\C-c")
    => (keymap (21 . reb-force-update) (2 . reb-change-target-buffer) ...)

Compare this to:

    (keymap-lookup reb-mode-map "C-c")
    => nil

Is the second result expected, or is it a bug?





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#52374: keymap-lookup with prefixes returns nil
  2021-12-08 12:21 bug#52374: keymap-lookup with prefixes returns nil Stefan Kangas
@ 2021-12-10 12:11 ` Lars Ingebrigtsen
  2021-12-10 13:33   ` Stefan Kangas
  0 siblings, 1 reply; 4+ messages in thread
From: Lars Ingebrigtsen @ 2021-12-10 12:11 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 52374

Stefan Kangas <stefan@marxist.se> writes:

> Run emacs -Q, type `M-x re-builder RET' and then `C-c C-q':
>
> Now evaluate:
>
>     (lookup-key reb-mode-map "\C-c")
>     => (keymap (21 . reb-force-update) (2 . reb-change-target-buffer) ...)
>
> Compare this to:
>
>     (keymap-lookup reb-mode-map "C-c")
>     => nil
>
> Is the second result expected, or is it a bug?

It's a bug.  Should be fixed now.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#52374: keymap-lookup with prefixes returns nil
  2021-12-10 12:11 ` Lars Ingebrigtsen
@ 2021-12-10 13:33   ` Stefan Kangas
  2021-12-10 13:34     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Kangas @ 2021-12-10 13:33 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 52374

Lars Ingebrigtsen <larsi@gnus.org> writes:

> It's a bug.  Should be fixed now.

Thanks!  Does this deserve a unit test?





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#52374: keymap-lookup with prefixes returns nil
  2021-12-10 13:33   ` Stefan Kangas
@ 2021-12-10 13:34     ` Lars Ingebrigtsen
  0 siblings, 0 replies; 4+ messages in thread
From: Lars Ingebrigtsen @ 2021-12-10 13:34 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 52374

Stefan Kangas <stefan@marxist.se> writes:

> Thanks!  Does this deserve a unit test?

Probably...  many of those smaller keymap-* functions were added in a
hurry while I was finishing up things and aren't well tested.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-12-10 13:34 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-08 12:21 bug#52374: keymap-lookup with prefixes returns nil Stefan Kangas
2021-12-10 12:11 ` Lars Ingebrigtsen
2021-12-10 13:33   ` Stefan Kangas
2021-12-10 13:34     ` Lars Ingebrigtsen

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).