Re: [PATCH 3/5] build: Add 'emacs-build-system'

[Alex Kost <alezost@gmail.com>]

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

Federico Beffa (2015-06-22 11:51 +0300) wrote:

> On Sun, Jun 21, 2015 at 10:40 PM, Alex Kost <alezost@gmail.com> wrote:
>>> new file mode 100644
>>> index 0000000..da2b594
>>> --- /dev/null
>>> +++ b/guix/build-system/emacs.scm
>> [...]
>>> +                    #:tests? ,tests?
>>> +                    #:phases ,phases
>>> +                    #:outputs %outputs
>>> +                    #:search-paths ',(map search-path-specification->sexp
>>> +                                          search-paths)
>>> +                    #:inputs %build-inputs)))
>>> +
>> spaces on this line ^
>
> Hi, thanks for the review!
>
> This and other spaces that you indicate as "extra", are included by
> Emacs by pressing TAB. This is because, as an example, the above
> snippet finishes an internal define form and not a top-level form.

Then don't press TAB on an empty line if you are not going to write
something there :-)

> I'm following the Emacs behavior. Is there a convention to suppress
> all spaces (in spite of what Emacs does)?

Well, these trailing spaces are artifacts of (inaccurate) coding.  Emacs
can't read user's mind to decide if the spaces are redundant or
intended.

As Mathieu, I also use 'delete-trailing-whitespace' command when I want
to get rid of the whitespaces in the current buffer.  You may try to put
it into 'after-save-hook', but I would not do this as I don't like when
my files are edited without asking me.

Also I prefer to _see_ the trailing whitespaces.  For that I set
'show-trailing-whitespace' variable to t in "text" and "prog" buffers:


[-- Attachment #2: show-whitespaces.el --]
[-- Type: application/emacs-lisp, Size: 199 bytes --]

[-- Attachment #3: Type: text/plain, Size: 10 bytes --]


-- 
Alex