false positive on 'new StdClass();'

Looking at code-style.pl again, it looks like I misinterpreted the rule (I previously read the error message and didn't really look at the regex). Is SomethingCapitalized OK, but somethingCapital not?

elsif (/([a-z])([A-Z])/) {
    $msg = "no mixed case function or variable names, use lower case and _";
  }

then shouldn't the Drupal coding standard be 'new stdclass();' - all lower case. I don't think php cares... I just tested on a php4 and php5 systems.

Ok, I'll add an exception. I've seen it on php.net as stdClass and StdClass. stdClass seems to be the standard. Do you think we should ignore both?

Should be fixed in the latest version!

Thanks!

Actually, probably better to pick one (you said stdClass is the most common on php.net?), update the coding standards accordingly, update the script so it only checks for that one, and submit patches to fix the inconsistencies.

That might be out of scope of this module though. ;)

Ok, between the facts that:

1. php.net calls it stdClass
2. There are more instances of stdClass than StdClass in core (by like one :P)
3. JavaScript is the closest model we have in Drupal, and it uses studlyCaps.

With Steven's ok, I've updated the coding standards to use "studlyCaps" on class names, and will roll a patch shortly. So if you could change it to only ok "stdClass" that would be cool. :)

Core patch committed: http://drupal.org/node/107015

Here's a patch for coder module to make it only exclude stdClass.

Thanks yched and webchick. I've checked in something that will flag StdClass, but not stdClass. I simplified the regex because we don't need to compare words that start as caps.