emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: New assoc-let package


From: Ted Zlatanov
Subject: Re: New assoc-let package
Date: Wed, 10 Dec 2014 14:00:44 -0500
User-agent: Gnus/5.130012 (Ma Gnus v0.12) Emacs/25.0.50 (gnu/linux)

On Wed, 10 Dec 2014 16:28:19 -0200 Artur Malabarba <address@hidden> wrote: 

AM> 2014-12-10 15:49 GMT+00:00 Ted Zlatanov <address@hidden>:
>> On Wed, 10 Dec 2014 10:43:23 -0500 Stefan Monnier <address@hidden> wrote:
>> 
>>>> Hmm, I don't see that the Package header is necessary.  At least
>>>> cfengine.el doesn't have it.  Am I missing something?
>> 
SM> You're probably right.
>> 
>> All right, when Artur gives the final OK, I'll merge to master. He's in
>> the ChangeLog already so his contributor papers are OK.

AM> Thanks Ted, there's only one small thing I'd like to add at the end of
AM> the header comments (quoted below).
AM> I've added it as an ammend to your branch, and pushed it to the
AM> malabarba/let-alist-package-rc3 branch.

AM> ;; Note that only one level is supported.  If you nest `let-alist'
AM> ;; invocations, the inner one can't access the variables of the outer
AM> ;; one.

I added something similar but yours is better :)

;; Note that only one level is supported.  You can't nest `let-alist'
;; invocations safely.

AM> If you're ok with that feel free to merge.

Yup, except I cherry-picked to avoid the unnecessary merge message.
Feel free to delete your RC branch--I deleted mine.

Ted




reply via email to

[Prev in Thread] Current Thread [Next in Thread]