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

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

Date: Wed, 27 Oct 2010 18:13:34 -0700

To: David Edmondson, notmuch@notmuchmail.org

Cc:

From: Carl Worth


On Thu, 29 Apr 2010 11:33:36 +0100, David Edmondson <dme@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@intel.com
part-000.sig (application/pgp-signature)

Thread: