search

abo-abo / lispy

Short and sweet LISP editing
http://oremacs.com/lispy/
1.21k stars 132 forks source link

Add autoload to most interactive functions #510

Closed rgrinberg closed 5 years ago

rgrinberg commented 5 years ago

lispyville in particular binds these to some keys. was there a reason why these autoloads were absent before?

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 63.436% when pulling 61e1137b00bab9f7a45dce5db24fa37640eaa613 on rgrinberg:autoload into 9f48176fe9a170848be0a07506d50e29b5f0dba3 on abo-abo:master.

abo-abo commented 5 years ago

was there a reason why these autoloads were absent before?

Yes. Autoloads are not free. When using package.el, the file lispy-autoloads.el is always loaded. The file by itself does nothing more than point that you need to (require 'lispy) after you do M-x lispy-mode.

The more autoloads a package has, the more LISP code that needs to be ran in lispy-autoloads.el. The only command that's reasonable to include in the M-x list is lispy-mode. All other code, like lispyville can simply say (require 'lispy).