defrepeater.el icon indicating copy to clipboard operation
defrepeater.el copied to clipboard

Compare with other packages in docs

Open jpkotta opened this issue 7 years ago • 1 comments

Nothing against this package, it seems well done. I used to use a similar function before I found and decided I liked easy-repeat (https://github.com/xuchunyang/easy-repeat.el).

jpkotta avatar Sep 05 '18 19:09 jpkotta

Thanks for sharing, I hadn't seen that, or the similar packages he links to.

I think that defrepeater's implemention (which I can take no credit for, as it was posted by Drew Adams) is much better and simpler. easy-repeat uses around advice for every command, which will interfere with other advice on those functions, likely confusing users with strange behavior until they track it down to multiply advised functions.

In contrast, the actual implementation of repetition in defrepeater is simply:

(let ((repeat-message-function #'ignore))
  (setq last-repeatable-command ,command)
  (repeat nil)))

And it does not interfere with the original commands.

I'll add a section to the readme comparing these packages. Thanks.

alphapapa avatar Sep 06 '18 00:09 alphapapa