Re: caps-mode.el

[Kevin Rodgers]

Kevin Rodgers wrote:
> Note this recommendation in define-minor-mode's doc string:
>
> | Optional KEYMAP is the default (defvar) keymap bound to the mode keymap.
> |   If it is a list, it is passed to `easy-mmode-define-keymap'
> |   in order to build a valid keymap.  It's generally better to use
> |   a separate MODE-map variable than to use this argument.
>
> So eliminate the :keymap LIST altogether:
...
> (defvar caps-mode-keymap
>   (let ((map (make-sparse-keymap)))
>     (mapc (lambda (char)
>             (define-key map 'caps-mode-self-insert-command))
>           "abcdefghijklmnopqrstuvwxyz")
>     map))

Oops, also from define-minor-mode's doc string:

| This function defines the associated control variable MODE, keymap MODE-map,
| toggle command MODE, and hook MODE-hook.

So that should be: (defvar caps-mode-map ...)

--
Kevin Rodgers