[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.
Ted