On Sat, 2011-12-24 at 11:56 +0000, Derick Rethans 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.
It's not only bad for merges, also confusing with blame/annotate while identifying the reason for bug. johannes > Derick > > -- > http://derickrethans.nl | http://xdebug.org > Like Xdebug? Consider a donation: http://xdebug.org/donate.php > twitter: @derickr and @xdebug > -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php