From: Daniel Hackney <dan@haxney.org>
To: Miles Bader <miles@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: The grand indentation unification project
Date: Sun, 27 May 2012 13:32:13 -0400 [thread overview]
Message-ID: <CAMqXDZti6sMmYbQ6f-imrH3RG1=+tK3yPMB_f1-qCG_n9G6bYA@mail.gmail.com> (raw)
In-Reply-To: <87mx4tbl7s.fsf@catnip.gol.com>
[-- Attachment #1: Type: text/plain, Size: 1337 bytes --]
I really don't want to waste any more of people's time, and I know this is
a very minor point, so please feel free to say "this isn't worth the
electrons it's printed on" and I'll let it go.
I do think having a uniform whitespace style (no spaces before tabs, etc.)
would be useful, but it isn't worth any real mental effort.
"Miles Bader" <miles@gnu.org> wrote:
>
> Daniel Hackney <dan@haxney.org> writes:
> > I understand not wanting to reawaken this bikeshedding thread, but I
> > would like to submit this humble one-liner which goes through all of
> > the elisp source and indents each file.
>
> Your original proposal (".dir-locals => indent-tabs-mode = t") seems
> fairly innocuous, as it won't, for the most part, change any line that
> isn't being changed already for some reason. So it would basically
> would result in minimal churn.
>
> _This_, on the other hand, would probably result in _massive_
> pointless code churn...
Yes, it would, but the idea would be to set `indent-tabs-mode' to t, add
`whitespace-cleanup to `before-save-hook', and have a single massive,
horrible, ugly "whitespace unification" commit, and then everything after
that follows the tab indent and whitespace styles. That way, you'd have one
big patch with all the whitespace changes, but future patches wouldn't have
any whitespace-only diffs.
[-- Attachment #2: Type: text/html, Size: 1658 bytes --]
next prev parent reply other threads:[~2012-05-27 17:32 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-26 19:45 The grand indentation unification project Daniel Hackney
2012-05-26 20:05 ` Glenn Morris
2012-05-27 9:28 ` Juri Linkov
2012-05-27 14:42 ` Daniel Hackney
2012-05-27 17:13 ` Miles Bader
2012-05-27 17:32 ` Daniel Hackney [this message]
2012-05-28 5:12 ` Stefan Monnier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAMqXDZti6sMmYbQ6f-imrH3RG1=+tK3yPMB_f1-qCG_n9G6bYA@mail.gmail.com' \
--to=dan@haxney.org \
--cc=emacs-devel@gnu.org \
--cc=miles@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.