[Top][All Lists]

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

Re: [Emacs-diffs] tzz/let-alist-package ab4667d: Add let-alist macro and

From: Ted Zlatanov
Subject: Re: [Emacs-diffs] tzz/let-alist-package ab4667d: Add let-alist macro and package.
Date: Wed, 10 Dec 2014 11:05:05 -0500
User-agent: Gnus/5.130012 (Ma Gnus v0.12) Emacs/25.0.50 (gnu/linux)

On Wed, 10 Dec 2014 10:47:36 -0500 Stefan Monnier <address@hidden> wrote: 

>> +;;; News:
>> +;;; Code:

SM> There's no need to have a ";;; News:" section, so if it's empty, better
SM> just remove it.

>> +(provide 'let-alist)
>> +;;; let-alist.el ends here
>> +
>> +
>> +;; Local Variables:
>> +;; sentence-end-double-space: t
>> +;; End:

SM> Either it ends there or it doesn't.  IOW, please move the file local
SM> variables earlier.

OK.  Both changes made in branch tzz/let-alist-package-rc2

It would be nice if user "tzz" was allowed to "git push -f" into
branches that start with "tzz".  That's probably hard to implement
though, so I hope you don't mind the new branches.  IMO it's better to
amend a commit than make a followup for a trivial fix.


reply via email to

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