I just wanted to work on a patch for #1515372: Caching Results, but soon stumbled over git diff showing dozens of whitespace changes in all files. As per the Drupal coding standards, I configured my editors to automatically remove trailing whitespace upon saving, but it seems there are a lot of trailing spaces in your files.
Could you please remove those to make contributing to your module easier? Attached is a patch, where I also removed the – now unneeded – $Id$ tags and fixed two typos in the very first function documentation comment.
Please review and commit!

(Your code also doesn't meet other parts of the standard, like comment line lengths, but these aren't as detrimental to participation as the whitespace issue. (Also I'm not one to talk there, as I haven't managed to update my modules in that respect, either.))

Comments

drunken monkey’s picture

StatusFileSize
new25.69 KB
drunken monkey’s picture

Just a note: there are also a lot of untranslated strings, mostly in the plugins. Didn't have the time to prepare a patch, though.

simon georges’s picture

Status: Needs review » Reviewed & tested by the community

Patch applies cleanly on current -dev.

michaelfavia’s picture

Status: Reviewed & tested by the community » Closed (fixed)

Committed with attribution. Thank you both.

[7.x-1.x f6b34bc]