Bug 66719 - check-webkit-style requires 2 spaces before comment in Python 1 in c++
Summary: check-webkit-style requires 2 spaces before comment in Python 1 in c++
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-22 14:58 PDT by Eric Seidel (no email)
Modified: 2019-05-02 16:24 PDT (History)
13 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2011-08-22 14:58:40 PDT 
Source/WebCore/rendering/InlineIterator.h:216:  One space before end of line comments  [whitespace/comments] [5]

check-webkit-style (due to PEP8) will tell me to do the opposite in my python code.

We should pick one.  I proposed PEP8 be damned and we standardize on 1-space everywhere.
Comment 1 David Levin 2011-08-22 15:13:12 PDT 
TLDR: I don't care either way.

I would be fine with using 1 space in python code (though I do like sticking with PEP8 as I think it makes it easier for people to do our style since it would be the standard style).

I honestly don't care too much about these issues (except for the fact that chromium disobeys PEP8 in naming and indents, which makes it harder for us to port code quickly and simply, but I lost that battle -- btw, if chromium followed PEP8 more closely, I would care more about this issue as it would make it uglier to port code).