Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files

Subject: Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files

Date: Fri, 13 Jan 2012 08:22:49 +0000

To: Xavier Maillard, Tomi Ollila, notmuch@notmuchmail.org

Cc: Tomi Ollila

From: David Edmondson

On Thu, 12 Jan 2012 23:07:03 +0100, Xavier Maillard <xavier@maillard.im> wrote:
> On Wed, 11 Jan 2012 17:13:35 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:
> > Executed
> >     for f in emacs/*.el
> >     do emacs -q --batch --eval \
> >         "(progn (find-file \"$f\") (whitespace-cleanup) (save-buffer))"
> >     done
> > 
> > which
> >     removed trailing whitespace and empty lines at the end of file
> >     removed spaces before tab (if any)
> >     converted 8 spaces to tabs in the beginning of line
> 
> +1
> 
> Maybe we should note something about this as a /coding style/ rule ?

How about "Running `indent-region' over the file should produce no
changes" as well?

• Previous message (by thread): Re: [PATCH 2/3] emacs: whitespace-cleanup and indent-region for emacs/*.el files

Thread:

• Tomi Ollila—[PATCH] emacs: whitespace-cleanup for emacs/*.el files [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Xavier Maillard—Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files [inbox, unread]
    • David Edmondson—Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files [inbox, signed, unread]
      • Tomi Ollila—Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files [inbox, unread]
        • David Edmondson—Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files [inbox, signed, unread]
          • Tomi Ollila—[PATCH 1/3] emacs/*.el: changed one-char comment prefix ';' to two: ';;' [inbox, notmuch::obsolete, notmuch::patch, unread]
            • Tomi Ollila—[PATCH 2/3] emacs: whitespace-cleanup and indent-region for emacs/*.el files [inbox, notmuch::obsolete, notmuch::patch, unread]
              • Austin Clements—Re: [PATCH 2/3] emacs: whitespace-cleanup and indent-region for emacs/*.el files [inbox, notmuch::review, unread]
                • Jameson Graef Rollins—Re: [PATCH 2/3] emacs: whitespace-cleanup and indent-region for emacs/*.el files [inbox, signed, unread]
                • Tomi Ollila—Re: [PATCH 2/3] emacs: whitespace-cleanup and indent-region for emacs/*.el files [inbox, unread]
                  • Austin Clements—Re: [PATCH 2/3] emacs: whitespace-cleanup and indent-region for emacs/*.el files [inbox, unread]
            • Tomi Ollila—[PATCH 3/3] RFC: batch-tools.el [inbox, notmuch::obsolete, notmuch::patch, unread]
            • David Edmondson—Re: [PATCH 1/3] emacs/*.el: changed one-char comment prefix '; ' to two: '; ; ' [inbox, signed, unread]
            • Austin Clements—Re: [PATCH 1/3] emacs/*.el: changed one-char comment prefix ';' to two: ';;' [inbox, notmuch::review, unread]
        • Xavier Maillard—Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files [inbox, unread]
          • David Bremner—Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files [inbox, unread]