[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] replace letf with cl-letf in org-mime
From: |
Nicolas Goaziou |
Subject: |
Re: [O] replace letf with cl-letf in org-mime |
Date: |
Wed, 01 Apr 2015 21:46:01 +0200 |
Eric Abrahamsen <address@hidden> writes:
> If I remove the two `eval's and treat "hook" like a normal variable, the
> call to `length' fails with:
>
> Wrong type argument: sequencep, org-mime-pre-org-hook
>
> So apparently `length' is seeing the symbol name, and not the symbol
> value.
Indeed.
> I tried changing the `let' to look like:
>
> (let ((hook (symbol-value (intern (....
What about (length (symbol-value hook)) instead?
> Here's a fixed version of the previous patch.
Thank you. Applied.
> I suppose I could also alter the "bhook" thing to use `symbol-value'
> instead of `eval', but that doesn't seem to be a net gain.
IMO, anything is a net gain compared to using `eval'.
Regards,
- Re: [O] replace letf with cl-letf in org-mime,
Nicolas Goaziou <=