[webkit-dev] Whitespace changes

Dan Bernstein mitz at apple.com
Wed Aug 26 22:43:11 PDT 2009


On Aug 26, 2009, at 10:35 PM, Oliver Hunt wrote:

> Adam just landed a fairly substantial patch that did nothing but  
> remove whitespace from the ends of lines.  While we had a thread  
> about style changes earlier, it was in the context of changes that  
> actually bring code into line with style guidelines in a way that  
> actually effects the visible layout of code.
>
> I do not believe that patches such as r47808 should be considered  
> valuable as the changes have no visible effect on code layout, but  
> do impact our ability to effectively use tools like svn or git  
> blame, however the situation is slightly ambiguous on this matter so  
> i thought i should bring it up on the mailing list.
>
> Does anyone else have thoughts on this?

While I do see value in cleaning up code, I think the cost of  
whitespace-only changes is higher than their benefit.


More information about the webkit-dev mailing list