This is the mail archive of the
gdb@sourceware.org
mailing list for the GDB project.
Re: Trailing Whitespace
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Phil Muldoon <pmuldoon at redhat dot com>
- Cc: gdb at sourceware dot org
- Date: Thu, 9 Jan 2014 19:01:45 +0400
- Subject: Re: Trailing Whitespace
- Authentication-results: sourceware.org; auth=none
- References: <CA+XNFZP6JzmOGo=nQDd4kab8RK1zvR4sij_zgS7qdPw7C1W9Vw at mail dot gmail dot com> <20140109144455 dot GA2972 at blade dot nx> <20140109145731 dot GA13720 at localhost dot localdomain>
> Also (and I am not sure if git can get around this somehow) whitespace
> cleanups tend to obliterate diffs/patches that were written before the
> cleanup took place. This makes merging a massive pain.
>
> So my 2 pence worth is, no, not a good idea ;)
Strongly seconded. Localized whitespace fixes are OK, because manageable
in the amount of work they generate, but please, not a massive cleanup.
And if we ever do that massive cleanup, I would request that a
pre-requisite is an "update" check on the git server that rejects pushes
of files violating that rule.
On my end of things, I configured my editor to simply highlight
trailing spaces, not strip them.
--
Joel