Bug 55253 – check-webkit-style should check for spaces after periods in comments

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Eric Seidel (no email)]

check-webkit-style should check for spaces after periods in comments

I feel like every few patches someone tells me my spaces after periods are wrong.  My typing teacher taught me 2, but I don't really care which.  But I don't want to have my patches cq-'d due to lack of "proper" spacing.  :)  I'd rather have check-webkit-style tell me what to do (assuming we even want to bother to specify it).

CCing folks who I believe care about this.

[Eric Seidel (no email)]

We should also add a rule to http://www.webkit.org/coding/coding-style.html if we're going to enforce this.

[Eric Seidel (no email)]

My tone in comment 0 sounds more angry than I mean. :)  I mean more to sound ignorant and willing to do what I"m told. :)

[David Levin]

(In reply to comment #2)
> My tone in comment 0 sounds more angry than I mean. :)  I mean more to sound ignorant and willing to do what I"m told. :)

I don't really care. I simply tell people because that is how I was brought up to do things in WebKit (per ap).

It is part of my way outstanding work to put this in the style guide as it is one of the items in
https://lists.webkit.org/pipermail/webkit-dev/2009-September/009807.html

I very much need to add all of that to the style guide (and to the style checker) -- I'll try to get to this (unfortunately these items are my side items to handle).

[David Levin]

Added ap because he cares/often notes it as well (but I don't think there is much debate here -- as this was already brought up on the alias and just hasn't made it into the style guide yet b/c no one like me has put it there).

[Darin Adler]

I do care about consistency in this. And we have chosen single spaces. So yes, lets put it in the style guide and the checker script.

[Alexey Proskuryakov]

I've been trained so by Darin (and I also prefer it because Russian typography uses uniform spacing, but that's completely irrelevant for WebKit).

[David Levin]

Created attachment 83906 [details]
Patch

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 83906 [details]:

inspector/debugger/debugger-pause-on-exception.html bug 55298 (authors: [e-mail] and [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 83906 [details]
Patch

Clearing flags on attachment: 83906

Committed r79787: <http://trac.webkit.org/changeset/79787>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.