From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Daniel Hackney Newsgroups: gmane.emacs.devel Subject: Re: The grand indentation unification project Date: Sun, 27 May 2012 13:32:13 -0400 Message-ID: References: <87mx4tbl7s.fsf@catnip.gol.com> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: multipart/alternative; boundary=f46d0418253e6397dc04c107fb4d X-Trace: dough.gmane.org 1338139948 28514 80.91.229.3 (27 May 2012 17:32:28 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Sun, 27 May 2012 17:32:28 +0000 (UTC) Cc: emacs-devel@gnu.org To: Miles Bader Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sun May 27 19:32:24 2012 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1SYhKI-0003ZR-BP for ged-emacs-devel@m.gmane.org; Sun, 27 May 2012 19:32:22 +0200 Original-Received: from localhost ([::1]:33628 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SYhKH-0006Nm-Pp for ged-emacs-devel@m.gmane.org; Sun, 27 May 2012 13:32:21 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:60158) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SYhKE-0006NG-Ct for emacs-devel@gnu.org; Sun, 27 May 2012 13:32:19 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SYhKC-0007cj-Hj for emacs-devel@gnu.org; Sun, 27 May 2012 13:32:17 -0400 Original-Received: from mail-wg0-f49.google.com ([74.125.82.49]:34205) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SYhKC-0007cU-3E for emacs-devel@gnu.org; Sun, 27 May 2012 13:32:16 -0400 Original-Received: by wgbds1 with SMTP id ds1so1809838wgb.30 for ; Sun, 27 May 2012 10:32:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=haxney.org; s=google; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=816aokKgRExdEro6G5QXtxDhoMxTLravanZT9QwssGQ=; b=j2ZSuik79acTQGfB4+o21bQcDh1AWJtagESgTP1L4R+lbOHxq8y98DdH42lAD7MWqI YDLVvzlDgxnDinh8Ab5B7dpxO6qxTQt2+NFTuN5c+opDG+Tn+me8cQ4N6e1Hm5Q7BAg5 P09wb4F59HhcxGd961vpGX3DXqikmYlX2VXn4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:x-gm-message-state; bh=816aokKgRExdEro6G5QXtxDhoMxTLravanZT9QwssGQ=; b=YmcK1gJe8PWOBW5As1XwtWi28vT8kOxXqDpn9c5+QMuQQ210nqAOkxm60a70RUmzjh x3xpPrEVSFavhkwm7fUR9q6LrNr9m/q1bzexiVKIKiviwhuKZCMAhzbrezVLtFCk0quM H66X5Irmt55IxDFndAmhxYG42FBsR/ORG9Y5bbjAFsbOPYQ73MFdMTcMXle33GBk+M1a UNmLH4lGOM7Kh/fdrmZYZ65WC/TWr+pd/mtTZymOhUjSQslGbmd4tEyetGeBDSB+GZ/F Wj+AUcC3Puqjde+z6IZgZSFq1Q2CVhBW9zILA39C8gkSK865iwufNh2RmOgZGHWk/waL U3fQ== Original-Received: by 10.180.84.6 with SMTP id u6mr2249625wiy.11.1338139933382; Sun, 27 May 2012 10:32:13 -0700 (PDT) Original-Received: by 10.180.87.167 with HTTP; Sun, 27 May 2012 10:32:13 -0700 (PDT) Original-Received: by 10.180.87.167 with HTTP; Sun, 27 May 2012 10:32:13 -0700 (PDT) In-Reply-To: <87mx4tbl7s.fsf@catnip.gol.com> X-Gm-Message-State: ALoCoQm5lS1JayPMTFxTEdyyDvc+l/NuWvp+umrBCUC3o+Wb1YWRFUoww9mALKkM9uCv83aXzth9 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 74.125.82.49 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:150661 Archived-At: --f46d0418253e6397dc04c107fb4d Content-Type: text/plain; charset=UTF-8 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" wrote: > > Daniel Hackney 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. --f46d0418253e6397dc04c107fb4d Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

I really don't want to waste any more of people's time, and I kn= ow this is a very minor point, so please feel free to say "this isn= 9;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.o= rg> wrote:
>
> Daniel Hackney <dan@haxney.org> writes:
> > I understand not wanting to reawaken this bikeshedding thread, bu= t 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 =3D> indent-tabs-mode =3D= t") seems
> fairly innocuous, as it won't, for the most part, change any line = that
> isn't being changed already for some reason. =C2=A0So it would bas= ically
> 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 massi= ve, horrible, ugly "whitespace unification" commit, and then ever= ything after that follows the tab indent and whitespace styles. That way, y= ou'd have one big patch with all the whitespace changes, but future pat= ches wouldn't have any whitespace-only diffs.

--f46d0418253e6397dc04c107fb4d--