From: Xavier Maillard <xma@gnu.org>
To: Tomi Ollila <tomi.ollila@iki.fi>
Subject: Re: [PATCH] emacs: whitespace-cleanup for emacs/*.el files
In-Reply-To: <yf639bkexs3.fsf@taco2.nixu.fi>
References: <1326294815-31926-1-git-send-email-tomi.ollila@iki.fi>
	<m2wr8w60e0.fsf@kcals.intra.maillard.im>
	<cunr4z4huzq.fsf@hotblack-desiato.hh.sledj.net>
	<yf639bkexs3.fsf@taco2.nixu.fi>
User-Agent: Notmuch/0.10.1 (http://notmuchmail.org) Emacs/23.2.1
	(i486-slackware-linux-gnu)
Date: Fri, 13 Jan 2012 23:27:11 +0100
Message-ID: <m2d3an5jcw.fsf@kcals.intra.maillard.im>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: notmuch@notmuchmail.org
Precedence: list
Reply-To: Xavier Maillard <xma@gnu.org>

On Fri, 13 Jan 2012 11:50:52 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:
> 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:

What about doing all of this directly in .dir-locals.el (could be
considered unsafe though to add an EVAL block).

/Xavier