Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:57085
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain lsces.co.uk from 213.123.20.127 cause and error)
Message-ID: <4EF5E7CB.20903@lsces.co.uk>
Date: Sat, 24 Dec 2011 14:55:07 +0000
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:7.0.1) Gecko/20110929 Firefox/7.0.1 SeaMonkey/2.4.1
MIME-Version: 1.0
To: PHP Internals <internals@lists.php.net>
References: <CABBJUperfGV7jqpjHw4sF7KvRSrbD-+Axe=153g-YYT111xEZg@mail.gmail.com> <CA+S5htNKQamqKQkBVRHqUN_UUHhwoTsBiNEUM1rwYC2FpaZ9-Q@mail.gmail.com> <CAEZPtU7o=e6kkwB2Mt9OW5irSk_CJbyXTJ5kpswf_XG_T1UU_Q@mail.gmail.com> <alpine.DEB.2.02.1112241155370.27751@whisky.home.derickrethans.nl> <CAEZPtU5Qkh-8HyRsubz1qpvG21rR5Qho7n1e0=L2eYYjffk7og@mail.gmail.com>
In-Reply-To: <CAEZPtU5Qkh-8HyRsubz1qpvG21rR5Qho7n1e0=L2eYYjffk7og@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [Proposal] Remove the tail ws in codes
From: lester@lsces.co.uk (Lester Caine)

Pierre Joye wrote:
>>> Laruence refers to the PHP C source code and removing trailing white
>>> >>  spaces is a good thing as it is part of our CS.
>>> >>
>>> >>  Go ahead Laruence:)
>> >
>> >  Please don't. Changing whitespace en-masse makes it a royal pain to do
>> >  merges later. Please just fix them when you change the code on that line
>> >  only.
> There is no deal or pain to fix them in all branches at once. There is
> no need either to do it all at once, but doing them while working on a
> file in a separate commit before the actual change is just fine and is
> a good practice.

I'm with Derek ... having commits that are just WS corrections can be irritating 
when tracking changes ... but it would most definitely better to get them fixed 
before moving to git which would lump mass commits like that together in a 
change set and make rolling back changes a nightmare.

Logically it makes a lot more sense simply to leave them alone until such time 
as a file is touched for other reasons ... and at that time then a separate 
commit of the WS prior to the bug fix IS good practice. Bundling a whole array 
of WS fixes across unrelated sections of code is not once you move to any DVCS 
system.

-- 
Lester Caine - G8HFL
-----------------------------
Contact - http://lsces.co.uk/wiki/?page=contact
L.S.Caine Electronic Services - http://lsces.co.uk
EnquirySolve - http://enquirysolve.com/
Model Engineers Digital Workshop - http://medw.co.uk//
Firebird - http://www.firebirdsql.org/index.php