[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq an
From: |
Clément Pit--Claudel |
Subject: |
Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item |
Date: |
Wed, 24 Aug 2016 11:13:25 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 |
On 2016-08-16 11:04, Stefan Monnier wrote:
>> Apparently the functions surrounding cl-fill use cl- prefix. so the
>> change appear to be more cosmetic than technical.
>
> Indeed. But since the "cl-" prefix has to be used for global (aka
> dynamically scoped) variables, it's good practice to *not* use it for
> lexically scoped variables.
>
> IOW, I'm all for such cosmetic changes that harmonize the code, but we
> should do that by removing (rather than adding) the "cl-" prefix where
> it's not needed.
Should the change be reverted, then?
signature.asc
Description: OpenPGP digital signature
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, (continued)
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Leo Liu, 2016/08/16
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Stefan Monnier, 2016/08/16
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, John Wiegley, 2016/08/16
- RE: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Drew Adams, 2016/08/16
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Stefan Monnier, 2016/08/16
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, John Wiegley, 2016/08/16
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Robert Weiner, 2016/08/16
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Lars Ingebrigtsen, 2016/08/16
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Tino Calancha, 2016/08/17
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Stefan Monnier, 2016/08/23
- Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item,
Clément Pit--Claudel <=
Re: [Emacs-diffs] master 50548fd: cl-fill: Rename arguments to cl-seq and cl-item, Tino Calancha, 2016/08/17