[prev in list] [next in list] [prev in thread] [next in thread] 

List:       webkit-dev
Subject:    [webkit-dev] Reformatting-only patches being applied to trunk
From:       Oliver Hunt <oliver () apple ! com>
Date:       2009-07-25 8:08:58
Message-ID: 7BD879DC-018C-4C64-96E6-05D73EEE7E32 () apple ! com
[Download RAW message or body]

I've just noticed that there have been a few purely style related  
patches being landed in the tree recently, I don't believe these are a  
good idea and that any further reformatting only patches be rejected.

Historically we have avoided purely style related changes as they  
impact our ability to track code changes efficiently and make patch  
merging more complex. The general approach to code cleanup is to clean  
up regions of code as we work in them -- this means that more-or-less  
the only code effected by reformatting is code that was being modified  
anyway.

Does anyone disagree with this approach?

--Oliver

_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
[prev in list] [next in list] [prev in thread] [next in thread]