[PATCH] emacs: Avoid runtime use of `cl'.

Carl Worth cworth at cworth.org
Wed Oct 27 18:13:34 PDT 2010


On Thu, 29 Apr 2010 11:33:36 +0100, David Edmondson <dme at dme.org> wrote:
> The GNU Emacs Lisp Reference Manual section D.1 says:
> 
> > *  Please don't require the cl package of Common Lisp extensions at
...
> Follow this advice, requiring the following changes where `cl' was
> used at runtime:

Thanks for this code, David. I was just sitting down to eliminate all
warnings in the compilation of notmuch, so this was very
convenient. I've committed this now.

I do think the code is a bit harder to read now, but I'm willing to let
maintainership of nasty bits of emacs code drift from me to you for
things like this. ;-)

And meanwhile, the fact that we now have a framework for automated
testing of emacs code makes me much more comfortable about changes like
this. (Not that I think that we necessarily have good coverage of the
code being tested here, but it's now a very small step from where we are
to adding that coverage.)

Thanks again,

-Carl

-- 
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20101027/cd6d331a/attachment.pgp>


More information about the notmuch mailing list