[webkit-dev] reminder: don't do blanket style fixups

David Levin levin at chromium.org
Wed Oct 19 19:19:56 PDT 2011


Trailing whitespace guidelines aren't part of the WebKit style guide on
purpose.

Thus, it wasn't a style clean-up, so I guess that would make it a change to
fit someone's preference.

dave

On Wed, Oct 19, 2011 at 6:38 PM, Ojan Vafai <ojan at chromium.org> wrote:

> I saw a patch get committed recently that just fixes trailing whitespace
> across a large swath of the codebase.
>
> In general, we prefer that pure style cleanups are only done as a precursor
> to actually modifying the code. Otherwise, while they serve to make the
> style consistent, they also make it considerably harder to dig through the
> commit history for a given chunk of code.
>
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20111019/a25b7cd5/attachment.html>


More information about the webkit-dev mailing list