emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Installation of bubbles.el


From: Stefan Monnier
Subject: Re: Installation of bubbles.el
Date: Tue, 28 Aug 2007 09:07:57 -0400
User-agent: Gnus/5.11 (Gnus v5.11) Emacs/22.1.50 (gnu/linux)

>  - i noticed some `define-key' forms at top-level.
>    the pattern is like so:
>      (defvar MAP INIT)
>      (define-key MAP ...)
>      (define-key MAP ...)
>      (define-key MAP ...) ...

>    i think it would be a good idea to move the defvars to the
>    beginning of the file, w/ value nil, and also consolidate
>    the define-key forms using the idiom:
>      (unless MAP
>        (let ((m INIT))
>          (define-key m ...) ...))

Better yet: move the define-keys directly inside the defvar.


        Stefan




reply via email to

[Prev in Thread] Current Thread [Next in Thread]