// PHP is really logical to have three different tokens for { with
    // inconsistent names and only one for a closing brace.

Ample comments and a test.

CommentFileSizeAuthor
php_is_braindead.patch1.6 KBchx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dries’s picture

Dries CreditAttribution: Dries commented

Waw, that is braindead. Should we open a bug report at php.net?

Crell’s picture

Crell CreditAttribution: Crell commented

Probably. But we need to work around it regardless.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Needs work

Committed to CVS HEAD.

chx’s picture

chx CreditAttribution: chx commented
Status: Needs work » Fixed

Wrong status IMO.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Issue tags: +PHPWTF

Tagging.

Status: Fixed » Closed (fixed)
Issue tags: -PHPWTF

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

Issue tags: +

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.