the code-style.pl script can't handle code containing the increment ++ operator.

CommentFileSizeAuthor
#4 code-style.patch669 byteskilles@www.drop.org
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

(not verified) CreditAttribution: commented

Need a regular expression genius to fix this. I can't figure it out so any other takers?

barryorourke’s picture

barryorourke CreditAttribution: barryorourke commented

I was running an older version of the script.

chx’s picture

chx CreditAttribution: chx commented
Component: Code » other

Well, I am using the latest version ($Id: code-style.pl,v 1.8 2003/12/28 10:40:17 dries Exp $) and the -- operator is not handled. So I have reopened this bug report.

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented
FileSize
code-style.patch669 bytes

Dunno if this causes elsewhere, but -- doesn't get reported anymore.

Bèr Kessels’s picture

Bèr Kessels CreditAttribution: Bèr Kessels commented

it works for me. itested it with some bogus $me++ and it reports no errors then.
+1 for this one-liner

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Fixed

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)