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.
We should also add a rule to http://www.webkit.org/coding/coding-style.html if we're going to enforce this.
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. :)
(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).
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).
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.
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).
Created attachment 83906 [details] Patch
The commit-queue encountered the following flaky tests while processing attachment 83906 [details]: inspector/debugger/debugger-pause-on-exception.html bug 55298 (authors: loislo@chromium.org and pfeldman@chromium.org) The commit-queue is continuing to process your patch.
Comment on attachment 83906 [details] Patch Clearing flags on attachment: 83906 Committed r79787: <http://trac.webkit.org/changeset/79787>
All reviewed patches have been landed. Closing bug.