Re: [jamboost] trailing whitespaces

From: David Abrahams (david.abrahams_at_[hidden])
Date: 2002-05-14 07:05:11

----- Original Message -----
From: "Vladimir Prus" <ghost_at_[hidden]>

> David Abrahams wrote:
> > ----- Original Message -----
> > From: "Vladimir Prus" <ghost_at_[hidden]>
> >
> > > Tabs are already banned from sources. What about banning trailing
> > > whitespace as well? They're not as evil as tabs, but I really feel
bad
> > > about something invisible in sources. Opinions?
> >
> > 1. Too hard to enforce
>
> Well, since I already run some scripts that checks for both tabs and
trailing
> whitespace before commit, this is easy for me.

Yes, but a "ban" forces that on others. And be sure you're not stripping
whitespace from within strings:

X = "
" ;

The trailing whitespace on the first line counts, FWIW.

> > 2. No serious practical consequences for having them there.
>
> Does that mean that nobody will object to my commits containing removal
of
> empty lines? Or does everybody run diff with "-b" option?

I didn't say there were not serious practical consequences to *removing*
them ;-). Actually I just think it's relatively harmless, but it provides
little benefit so it shouldn't be forced on developers.

-Dave