madrobby/zepto — pull request #662 by dmp42 (2012-12-14 21:01:57+00:00)
Title: Various fixes

This contain a (still reasonable I hope) number of fixes - if you would prefer a separate pull request per change, please say so I'll do - I thought it would be easier to first discuss them (if need be) in one place rather than a dozen, then possibly amend the pull request.

FWIW I ran the tests ok.

Content goes by order from "trivial" to "possibly need a discussion":
- comment clarifying eval scope https://github.com/jsBoot/zepto/commit/3b207b868d7b79ef1540605ba7e9346268f7208d
- useless redefine of already defined variables: https://github.com/jsBoot/zepto/commit/0c5f9db8a049ac73a024a979fa76ef6deb143fae
- out of scope variables use https://github.com/jsBoot/zepto/commit/a6524b62770b2cce47386d0de9ed0da19d8ab083
- always declared function fix https://github.com/jsBoot/zepto/commit/f233b2f07119daab020c57a4f7ec33d677dc7d3f
- more precise comparisons with 0 (== 0 --> === 0 / != 0 ---> !== 0) https://github.com/jsBoot/zepto/commit/720412604ac75162975961baa9f5740ba9ced9a2 and https://github.com/jsBoot/zepto/commit/031d718b6d6b992fe96c1802a1eac1d4ea67ac44
- optimization by not creating functions at each iteration https://github.com/jsBoot/zepto/commit/2bee5cc9527db79585deb018c55ce75a50bc2504
- use local for variables consistently: https://github.com/jsBoot/zepto/commit/4526ef9d274bb4e0695b235ae9431efa98fe1c9c
- arguments.callee removal: https://github.com/jsBoot/zepto/commit/e55060792be2a6634687f5c56fa0a9f862d119e3 and https://github.com/jsBoot/zepto/commit/9efbfd50382722e3cdb43adad1075973c8f5a0e0 - the reasoning being to not break strict mode if possible

In the hope this helps.
Thanks a lot!