The attached patch conforms *all* the sentence spacing in code comments to a single space following the period. Note that without this patch, core is very inconsistent; it mostly conforms to this standard but the size of this patch no doubt indicates how often we deviate from that standard. If this is committed, I'll edit the coding standards page and add an entry for one-space sentence spacing.

Please, please, someone commit this path before it no longer applies. If anything is changed in here other than sentence spacing, it was a mistake.

CommentFileSizeAuthor
#2 coding-standards-spacing-change.patch1.06 KBkeith.smith
no-more-double-spaces.patch145.51 KBkeith.smith

Comments

dries’s picture

dries commented
Status: Needs review » Fixed

Welp, many files touched! Committed to CVS HEAD. Thanks.

keith.smith’s picture

keith.smith commented
Status: Fixed » Needs review
StatusFileSize
new coding-standards-spacing-change.patch1.06 KB

Now that core is fixed, does anyone have an issue with this patch to CODING_STANDARDS.html?

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me.

keith.smith’s picture

keith.smith commented
Status: Reviewed & tested by the community » Fixed

http://drupal.org/cvs?commit=160434

CODING_STANDARDS.html patched to include this standard.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.