[webkit-dev] Whitespace changes
Mark Rowe
mrowe at apple.com
Thu Aug 27 13:37:46 PDT 2009
On 2009-08-27, at 12:57, Dirk Pranke wrote:
> On Thu, Aug 27, 2009 at 11:55 AM, Peter Kasting<pkasting at google.com>
> wrote:
>> On Wed, Aug 26, 2009 at 10:43 PM, David Levin <levin at chromium.org>
>> wrote:
>>>
>>> fwiw, I know that the check-webkit-style checks for trailing
>>> whitespace
>>> (and I approved that change - sorry), but I think it should
>>> probably be
>>> removed.
>>
>> Since opinions were requested, I'm in favor of both the check-
>> webkit-style
>> change here and Adam's patch, as well as other cleanup patches.
>> Maintaining
>> a cultural attitude that is widely positive towards cleanup makes
>> people
>> feel less reticent about cleaning up, and taking ownership of, code;
>> frowning on certain types of cleanup makes people less likely to do
>> _any_
>> cleanup.
>> As far as blame annotations, I don't mind paging past a cleanup
>> change when
>> trying to do archaeology on code; I have to go past enough other
>> changes
>> already that one more doesn't make much of a difference overall.
>
> +1
>
> I see little point in having coding standards if you don't encourage
> people to use them. There is enough churn in the tree that there are
> already a large number of changes to skip over, so skipping over
> reformatting (even whitespace-only) seems like a small burden to me.
There's a difference between enforcing our coding style on new code,
and retroactively applying our new style to the hundreds of thousands
of lines of code that already exist. Your comment conflates these two
issues. No-one is going to disagree that we should enforce the
coding style on new code. That would be stupid. However it is clear
that applying our current coding style to existing code is a much less
clear-cut issue. We wouldn't be having this discussion if it were as
obvious as your comment suggests.
There's also a huge difference between enforcing WebKit coding style
and enforcing some other coding style. Trailing whitespace is not
mentioned in the WebKit coding style guidelines.
- Mark
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3166 bytes
Desc: not available
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20090827/0e60719d/attachment.bin>
More information about the webkit-dev
mailing list