search

emacs-helm / helm-descbinds

A helm frontend for describe-bindings.
GNU General Public License v3.0
116 stars 12 forks source link

autoload command helm-descbinds #1

Closed tarsius closed 12 years ago

tarsius commented 12 years ago

So that library helm-descbinds' only has to be loaded if the command is actually used. If the user chooses to depend on the autoload he has to bind the command manually.

cofi commented 12 years ago

I don't think calling helm-descbinds directly is the main usecase. Especially if you replace only the C-h b entry point and leave PREFIXKEY C-h to the original describe-bindings.

But if you see any value here I'll merge it anyway.

tarsius commented 12 years ago

Yes, you are right that is more useful. I edited the commit to keep the autoload but remove the change in the Commentary.

I think there is value in that - at the same time it avoids confusing users.

cofi commented 12 years ago

By applying the same logic the more useful autoload cookie would then be at helm-descbinds-install, right?

tarsius commented 12 years ago

Don't know about more useful, but useful too.

On Sun, Mar 18, 2012 at 22:58, Michael Markert reply@reply.github.com wrote:

By applying the same logic the more useful autoload cookie would then be at helm-descbinds-install, right?

Reply to this email directly or view it on GitHub: https://github.com/emacs-helm/helm-descbinds/pull/1#issuecomment-4564832