If I am to hack on code, I like it looking decent. So tonight I've embarked on a journey for to make the Mollom module's code more Drupal code style conformant. ;)

CommentFileSizeAuthor
#1 295872_code_style_conformance-1.patch47.88 KBFreso

Comments

Freso’s picture

Freso commented
Status: Active » Needs review
StatusFileSize
new 295872_code_style_conformance-1.patch47.88 KB
  • Lots of whitespace and indentation fixes.
  • A few else if to elseif.
  • A few double quote to single quote changes.
  • A strlen() to drupal_strlen().
  • A db_query('... LIMIT 1000') to db_query_range('...', 0, 1000).
  • A Doxygen fix, /* to /**.
  • Added @file block to .module, .install, and .test.

End result? Cleaner code and a lot less notices, warnings, and errors from Coder module. :)

dries’s picture

dries commented
Status: Needs review » Fixed

Good catch. Committed to CVS HEAD. Thanks!

Anonymous’s picture

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

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