Re: Cleaning up dangling whitespace
| From: | Pierre Joye | 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