[PATCH] emacs: Fix "not defined at runtime" warning
Tomi Ollila
tomi.ollila at iki.fi
Thu Aug 9 00:46:56 PDT 2012
On Thu, Aug 09 2012, Austin Clements <amdragon at MIT.EDU> wrote:
> Previously, the Emacs byte compiler produced the warning
>
> the function `remove-if-not' might not be defined at runtime.
>
> because we only required cl at compile-time (not runtime). This fixes
> this warning by requiring cl at runtime, ensuring that the definition
> of remove-if-not is available.
> ---
LGTM.
It took me a while to understand this. Now I just don't understand
how did this work before. And why (require 'cl) is enough as
remove-if-not is defined in cl-seq.el -- cl-seq.el requires cl
but not vice-versa -- or at least I don't see that. Anyway
(require 'cl) is the common way.
Tomi
> emacs/notmuch-lib.el | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el
> index 30db58f..900235b 100644
> --- a/emacs/notmuch-lib.el
> +++ b/emacs/notmuch-lib.el
> @@ -24,7 +24,7 @@
> (require 'mm-view)
> (require 'mm-decode)
> (require 'json)
> -(eval-when-compile (require 'cl))
> +(require 'cl)
>
> (defvar notmuch-command "notmuch"
> "Command to run the notmuch binary.")
> --
> 1.7.10
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list