Re: Trailing Whitespace

Gary Benson <[email protected]>
Newsgroups gmane.comp.gdb.devel
Message-ID <[email protected]>
Ben Longbons wrote:
> I have my editor set to trim all trailing whitespace, because
> trailing whitespace is never desirable.
> 
> However, since other people have in times past committed code that
> *does* contain trailing whitespace, this creates noise when I'm
> trying to create a patch.
> 
> Therefore, I suggest that someone create a commit that removes all
> the trailing whitespace. The pressing need is to apply this to gdb/
> but there is no reason it couldn't be applied to other parts if need
> be.
> 
> A commit that does nothing besides change trailing whitespace is
> *not* harmful to the history, because git-blame and other tools can
> all ignore whitespace.
> 
> Therefore, the only consideration is whether this will cause
> problems for people with changes that are not yet committed in the
> main repo.  The script at the end of this message demonstrates how
> to avoid problems with all 4 of the local branch strategies that I
> know about.

I see nobody replied to this, but it would be pretty nice to have the
repo cleaned in this way.

Does anybody have any reason not to do this?

Thanks,
Gary

-- 
http://gbenson.net/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.