Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:57081
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.213.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <alpine.DEB.2.02.1112241155370.27751@whisky.home.derickrethans.nl>
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>
Date: Sat, 24 Dec 2011 15:33:14 +0100
Message-ID: <CAEZPtU5Qkh-8HyRsubz1qpvG21rR5Qho7n1e0=L2eYYjffk7og@mail.gmail.com>
To: Derick Rethans <derick@php.net>
Cc: Laruence <laruence@php.net>, Rafael Kassner <kassner@gmail.com>, 
	PHP Internals <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] [Proposal] Remove the tail ws in codes
From: pierre.php@gmail.com (Pierre Joye)

On Sat, Dec 24, 2011 at 12:56 PM, Derick Rethans <derick@php.net> wrote:
> On Sat, 24 Dec 2011, 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.

-- 
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org