Re: [jamboost] trailing whitespaces

From: Vladimir Prus (ghost_at_[hidden])
Date: 2002-05-14 09:58:25

David Abrahams wrote:

> > > 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.

Okay.

> > > 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.

Okay. I'll be removing them from the file I commit. 

- Volodya