This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Fix whitespace in comments.

From: Tobias Burnus <burnus at net-b dot de>
To: Thomas Koenig <tkoenig at netcologne dot de>, VandeVondele Joost <joost dot vandevondele at mat dot ethz dot ch>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>
Date: Sat, 20 Sep 2014 16:48:13 +0200
Subject: Re: [PATCH] Fix whitespace in comments.
Authentication-results: sourceware.org; auth=none
References: <908103EDB4893A42920B21D3568BFD9315105FDE at MBX13 dot d dot ethz dot ch> <541D50BA dot 6030906 at netcologne dot de>

Hi Joost and Thomas,

On 20.09.2014 12:02, Thomas Koenig wrote:

A somewhat trivial patch to cleanup whitespace issues in comments: sed "s/\. \*\//\.  \*\//g"
Tested with a recompile only. Ok for trunk ?

OK.  (Obvious, really).

I want to point a downside of massive code changes: It makes it moredifficult to find out with "git/svn blame" when or why a line of codewas changed.

That's not so much a problem of comment changes like this one – and I amfine with that change. :-)

However, I wanted to point out that seemingly trivial and obviouslycorrect patches can have a downside. (One can still do such changes, butat least one should have weighted them against the downside.)


Tobias

Follow-Ups:
- RE: [PATCH] Fix whitespace in comments.
  - From: VandeVondele Joost
- Re: [PATCH] Fix whitespace in comments.
  - From: Jeff Law

References:
- [PATCH] Fix whitespace in comments.
  - From: VandeVondele Joost
- Re: [PATCH] Fix whitespace in comments.
  - From: Thomas Koenig

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]