Re: [Annotation] New package: smart-yank.el

[Stefan Monnier <monnier@iro.umontreal.ca>]

> I intend to upload the attached package "smart-yank.el" to Gnu Elpa
> (does that work again btw?).
> Any comments, objections, ...or something like that?  Thanks.

Sounds good to me.  See nitpicks below.


        Stefan


> (let ((r (smart-yank--stopwatch)))
>   (defun smart-yank-yank-pop (&optional arg)
[...]
> (declare-function smart-yank-yank-pop 'smart-yank)

An arguably cleaner way to get the same result would be:

   (defalias 'smart-yank-yank-pop
     (let ((r (smart-yank--stopwatch)))
       (lambda (&optional arg)
         "blabla"
         ...)))

> (define-minor-mode smart-yank-mode
[...]
>   :global t :keymap smart-yank-map

Please name the keymap `smart-yank-mode-map' and don't use the :keymap
argument at all.