The attached patch fixes a whole bunch (262 to be precise) of coding standards issues with hotkey. Following this patch it now gets a clean bill of health from coder module. I haven't (that I know of) made any logic changes at all, this is pretty much all just correcting indentation and code style.

This would be a pain to have to reroll, so if we could get this in before making any other commits to hotkey that would be great :)

CommentFileSizeAuthor
hotkey_coding_standards.patch26.45 KBowen barton

Comments

shawn conn’s picture

Status: Needs review » Reviewed & tested by the community

Yeah, my IDE was using tabs instead of double spaces for indention with my realization. This, and other minor changes, will be rolled out in the next commit. Thanks!

shawn conn’s picture

Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

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