[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Reformat all of src/
From: |
Eli Zaretskii |
Subject: |
Re: Reformat all of src/ |
Date: |
Thu, 29 Dec 2016 05:41:18 +0200 |
> From: Richard Stallman <address@hidden>
> Date: Wed, 28 Dec 2016 20:36:40 -0500
> Cc: address@hidden, address@hidden
>
> One can argue about whether it was worth time to make the changes, but
> since that has been done already, why not put it in?
>
> A single change which affects only whitespace
> won't cause any confusion no matter how many places that whitespace
> is changed.
It makes digging into code history harder, especially when files are
also renamed later.
We arrived at the current convention of not doing such changes unless
real code changes are also committed based on our practice, so I don't
think we should reverse that decision.
- Re: Reformat all of src/, (continued)
- Re: Reformat all of src/, Phillip Lord, 2016/12/29
- Re: Reformat all of src/, Eli Zaretskii, 2016/12/29
- Re: Reformat all of src/, John Covici, 2016/12/29
- Re: Reformat all of src/, John Covici, 2016/12/29
- Re: Reformat all of src/, John Yates, 2016/12/28
Re: Reformat all of src/, Richard Stallman, 2016/12/28
Re: Reformat all of src/, Stefan Monnier, 2016/12/29
Reformat all of src/, hector, 2016/12/28
Re: Reformat all of src/, Chris Gregory, 2016/12/28