multi-touch-dev team mailing list archive
-
multi-touch-dev team
-
Mailing list archive
-
Message #00969
Re: Policy on formatting fixes intermixed with code changes?
On 03/27/2012 06:10 AM, Daniel d'Andrada wrote:
> On 26/03/12 18:09, Chase Douglas wrote:
>> Hi all,
>>
>> As the uTouch project is growing larger and larger, we are now starting
>> to hit issues where code formatting may stray from the unity style
>> guidelines. When we develop changes, we might see incorrect style in
>> surrounding code. Should we have a policy that requires code format
>> fixes to be contained within separate commits? This has come up as an
>> issue in a recent merge proposal, and we haven't really discussed it
>> previously.
>>
>> [...]
> I suggest only fixing coding style in the lines your commit is going
> change anyway and not doing changes that solely fix coding style at all,
> even if they come as a separate commit. Unless something really ugly or
> big has happened (e.g. an entire file is using tabs instead of spaces).
Why not?
Follow ups
References