General
- Are all vars used somewhere?
- Do the vars have properly cased, and meaningful names?
- Style looks appropriate
- No unnecessarily duplicated logic
- Code intent is clear upon initial reading
- Any code that isn't clear, but is needed, has an effective comment
- Are method calls or attribute lookups every called on entities that could be undefined/null?
- The functions can appropriately handle unexpected inputs.
- Commented code has been removed (comments themselves are fine).