01org/appframework — pull request #204 by Philzen (2013-04-13 18:15:57+00:00)
Title: Code cleanup, js doc, minor scoping fixes

While trying to understand something behind the scenes i stumbled across the code where the "activeDiv" public property was declared twice in jq.ui.js. So i went a little further down the rabbit hole...

In general the code hinting bar in Netbeans 7.3 makes eyes bleed with the jqui-source opened. So i went through it reduced the warning count. Only issue left is the prominent usage of the type-agnostic comparison operators - would be great if those could be diminished going forward, [as according to jsperf](http://jsperf.com/equality-operators-with-the-same-types/4) a type-specific comparison is always slightly faster and generally makes more robust code imho.

Kind of changes done:
- fixed some (clearly unambiguous) variables to stay in their scope
- corrected / completed JS Doc
- removed a few unused function scoped variables
- added numerous EOL semicolons