Re: Cleaning up dangling whitespace

From: Date: Tue, 06 Jan 2015 04:56:54 +0000
Subject: Re: Cleaning up dangling whitespace
References: 1  Groups: php.internals 
Request: Send a blank email to internals+get-80220@lists.php.net to get a copy of this message
On Mon, Jan 5, 2015 at 2:58 PM, Stanislav Malyshev <smalyshev@gmail.com> wrote: > Hi! > > The code in the PHP source repo has a lot of random dangling whitespace > at the end of lines, which looks sloppy and produces some weird effects > in diffs because some editors auto-clean it and some do not. I'd like to > clean it up (it's easy to do automatically) in master, but this produces > a big patch that may also create some conflicts with existing pull reqs > (easily resolved, but still). > So, I'd like to ask if there are any objections to such cleanup? If not, > then I'll do it, probably next weekend. Thanks for asking :) And go ahead, they should not be there in the 1st place, we have CS for a reason :) That reminds me something I was wondering, would it be possible to have a kind of CS check on push? :) Cheers, -- Pierre @pierrejoye | http://www.libgd.org

« previous php.internals (#80220) next »