Re: master 7b24460: * lisp/let-alist.el: Cosmetic changes.

Re: master 7b24460: * lisp/let-alist.el: Cosmetic changes.

[Ted Zlatanov]

On Wed, 10 Dec 2014 21:18:53 +0000 Glenn Morris <rgm@gnu.org> wrote: 

GM>     * lisp/let-alist.el: Cosmetic changes.
GM> -;; Local Variables:
GM> -;; sentence-end-double-space: t
GM> -;; End:
GM> -
GM>  ;;; let-alist.el ends here

Why remove this?  It seemed like a reasonable local override, but I'm OK
with removing it if that's how it is.

Ted

Re: master 7b24460: * lisp/let-alist.el: Cosmetic changes.

[Glenn Morris]

Ted Zlatanov wrote:

> GM> -;; Local Variables:
> GM> -;; sentence-end-double-space: t
> GM> -;; End:
> GM> -
> GM>  ;;; let-alist.el ends here
>
> Why remove this?

Because that's both the default value, and the setting from
.dir-locals.el, so re-specifying the same value in an individual file is
just clutter.

Re: master 7b24460: * lisp/let-alist.el: Cosmetic changes.

[Ted Zlatanov]

On Wed, 10 Dec 2014 21:00:11 -0500 Glenn Morris <rgm@gnu.org> wrote: 

GM> Ted Zlatanov wrote:
GM> -;; Local Variables:
GM> -;; sentence-end-double-space: t
GM> -;; End:
GM> -
GM> ;;; let-alist.el ends here
>> 
>> Why remove this?

GM> Because that's both the default value, and the setting from
GM> .dir-locals.el, so re-specifying the same value in an individual file is
GM> just clutter.

Oh, I see.

This package will also be maintained outside the Emacs core. It would
make life eaiser if the file could be the same in both places. Could you
reconsider?

Thanks
Ted

Re: master 7b24460: * lisp/let-alist.el: Cosmetic changes.

[Ted Zlatanov]

On Thu, 11 Dec 2014 06:55:58 -0500 Ted Zlatanov <tzz@lifelogs.com> wrote: 

TZ> On Wed, 10 Dec 2014 21:00:11 -0500 Glenn Morris <rgm@gnu.org> wrote: 
GM> Ted Zlatanov wrote:
GM> -;; Local Variables:
GM> -;; sentence-end-double-space: t
GM> -;; End:
GM> -
GM> ;;; let-alist.el ends here
>>> 
>>> Why remove this?

GM> Because that's both the default value, and the setting from
GM> .dir-locals.el, so re-specifying the same value in an individual file is
GM> just clutter.

TZ> Oh, I see.

TZ> This package will also be maintained outside the Emacs core. It would
TZ> make life eaiser if the file could be the same in both places. Could you
TZ> reconsider?

Um, never mind, Artur removed those local vars from the package he
pushed into the GNU ELPA.

Ted

Re: master 7b24460: * lisp/let-alist.el: Cosmetic changes.

[Stefan Monnier]

> This package will also be maintained outside the Emacs core. It would
> make life eaiser if the file could be the same in both places. Could you
> reconsider?

I think it's perfectly fine to keep such redundant settings, of course.
But, if you re-add it, please check with C-u M-x checkdoc-current-buffer
that the result follows the formatting convention.


        Stefan

Re: master 7b24460: * lisp/let-alist.el: Cosmetic changes.

[Artur Malabarba]

[-- Attachment #1: Type: text/plain, Size: 556 bytes --]

We can leave them out now.
I've setup my configs accordingly so I don't mind anymore.
On 11 Dec 2014 13:24, "Stefan Monnier" <monnier@iro.umontreal.ca> wrote:

> > This package will also be maintained outside the Emacs core. It would
> > make life eaiser if the file could be the same in both places. Could you
> > reconsider?
>
> I think it's perfectly fine to keep such redundant settings, of course.
> But, if you re-add it, please check with C-u M-x checkdoc-current-buffer
> that the result follows the formatting convention.
>
>
>         Stefan
>
>

[-- Attachment #2: Type: text/html, Size: 872 bytes --]