Re: [lwip-members] whitespace and CVS commits/diffs

[Kieran Mansley]

On Fri, 14 Feb 2003 address@hidden wrote:

> Hello committers,
>
>
> the combination of everyone using different white-space settings,
> along with CVS checkins of changed whitespace, really makes it hard
> to peer-review changes made by others.
>
> Search on "tabs" in the lwip-members archive, we had a small
> topic on this earlier.
>
> - The idea is to use "spaces only, 2 spaces per indent".
> - Do not allow your editor to change tabs and spaces automatically.
> - Make sure your editor inserts spaces for each tab you press.
>
> This, and possible committing only non-whitespace changes in CVS,
> will really allow us to diff between revisions in a sane manner.
>
> Is this satisfactory with everyone else, or do people prefer tabs?

No, what you suggest sounds good.  In addition, I always run any files
that are flagged for checkin through ediff to ensure that I don't make any
whitespace changes.  Others might like to try something similar.

Kieran