[webkit-dev] Reformatting-only patches being applied to trunk

Ojan Vafai ojan at chromium.org
Sat Jul 25 16:11:24 PDT 2009

On Sat, Jul 25, 2009 at 4:39 PM, David Hyatt <hyatt at apple.com> wrote:

> On Jul 25, 2009, at 3:08 AM, Oliver Hunt wrote:
>  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.
> I completely disagree.  I see nothing wrong with patches that are purely
> style cleanup.  Someone doesn't have to be doing significant work in an area
> of code to fix bad style in that code.  We've had a problem with bad style
> lingering in some files precisely because the area of code isn't being
> touched much.  It would be great to see those files cleaned up.

One other advantage of getting most or all of the code base to have the
correct style is that we could hook up the new linting tool to an svn/git
precommit hook so that new style errors don't get introduced. We could also
hook it into any create-patch scripts so that patches (new) contributors
upload don't need to be manually vetted by a reviewer for style. Eventually,
when we start work on a code review tool, we could automatically flag any
style violations without reviewer oversight.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20090725/7089d4b3/attachment.html>

More information about the webkit-dev mailing list