NEW 66719
check-webkit-style requires 2 spaces before comment in Python 1 in c++ 
https://bugs.webkit.org/show_bug.cgi?id=66719
Summary check-webkit-style requires 2 spaces before comment in Python 1 in c++
Eric Seidel (no email)
Reported 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.
Attachments
Add attachment proposed patch, testcase, etc.
David Levin
Comment 1 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).
Note You need to log in before you can comment on or make changes to this bug.