Skip to content
This repository was archived by the owner on Aug 23, 2025. It is now read-only.

Conversation

jyp
Copy link

@jyp jyp commented Jul 20, 2023

This fixes #1032, which also provides background.

With this change, use-package p puts in scope all the symbols declared by package p in scope of its :config block. The tradeoff is that init is slightly slower if interpreted. Compiling init.el gives the same runtime behavior as previously.

Perhaps there is a way to change use-package--eval-when-compile to obtain both fewer warnings at compile time and best runtime complexity, but this is beyond my elisp understanding.

With this change, `use-package p` puts in scope all the symbols
declared by package p in scope of its :config block. The tradeoff is
that init is slightly slower *if interpreted*. Compiling init.el gives
the same runtime behaviour as previously.

Fixes jwiegley#1032
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Byte compilation warning "the function ‘js2-imenu-extras-setup’ might not be defined at runtime."
1 participant