From: Tomi Ollila <tomi.ollila@iki.fi>
To: David Edmondson <dme@dme.org>, Xavier Maillard <xma@gnu.org>,
	notmuch@notmuchmail.org
Subject: Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files
In-Reply-To: <cunr4z4huzq.fsf@hotblack-desiato.hh.sledj.net>
References: <1326294815-31926-1-git-send-email-tomi.ollila@iki.fi>
	<m2wr8w60e0.fsf@kcals.intra.maillard.im>
	<cunr4z4huzq.fsf@hotblack-desiato.hh.sledj.net>
User-Agent: Notmuch/0.10.2+176~g3f9d738 (http://notmuchmail.org) Emacs/23.3.1
	(i686-pc-linux-gnu)
Date: Fri, 13 Jan 2012 11:50:52 +0200
Message-ID: <yf639bkexs3.fsf@taco2.nixu.fi>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Tomi Ollila <tomi.ollila@iki.fi>
Precedence: list

On Fri, 13 Jan 2012 08:22:49 +0000, David Edmondson <dme@dme.org> wrote:
> 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?

So, In addition to 'uncrustify.cfg' we need emacs reformatter along lines:

(defun reformat-elisp-buffer ()
  (interactice)
  (emacs-lisp-mode)
  (whitespace-cleanup)
  (indent-region (point-min) (point-max)))

And this located in same directory as 'uncrustify.cfg', wherever
that ends up.

Tomi