GNU bug report logs - #25187
Initialization of `shell-mode-map' is not safe

Previous Next

Full log

View this message in rfc822 format

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Alexander Shukaev <emacs <at> Alexander.Shukaev.name>
Cc: 25187 <at> debbugs.gnu.org
Subject: bug#25187: Initialization of `shell-mode-map' is not safe
Date: Wed, 09 Oct 2019 09:24:15 +0200

Alexander Shukaev <emacs <at> Alexander.Shukaev.name> writes:

> consider
>
> (defvar shell-mode-map
>   (let ((map (nconc (make-sparse-keymap) comint-mode-map)))

[...]

> from 'shell.el'.  When somebody decides to customize `comint-mode-map'
> by, for example, first wiping it out and then filling it again,
>
> (with-eval-after-load 'comint	
>   ;; Wipe `comint-mode-map':
>   (setcdr comint-mode-map nil)
>   ;; Fill `comint-mode-map':
>   ...)
>
> then when the `shell' feature is loaded, it will inevitably fail
> because `(lookup-key comint-mode-map [menu-bar completion])' may no
> longer return a keymap but rather nil or a number.

I've now fixed this in Emacs 27 by just removing the nconc from the
shell mode map -- shell-mode is derived from comint-mode, so it
automatically inherits its map.

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

Previous Next

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