aredridel/html5 — pull request #58 by yourcelf (2012-05-10 17:54:13+00:00)
Title: more var keywords, and two variable name errors

Here are a few more var keywords, as well as two presumed variable name errors:

In `lib/html5/parser/in_body_phase.js`, line 496, I think it wants "this.endTagOther" and not "endTagOther".

In `lib/html5/tokenizer.js` line 385, I think it wants "current_token.name" and not "current_token_name".

It looks like we don't have test coverage for those two, but before/after produce the same test result counts (891 tests pass, 447 tests fail).

p.s. I'm adding these because the Mocha test framework by default complains about global variable leaks, which makes my tests fail if I link html5. I'm using jshint to find the vars. Using this config for jshint:

jshint.conf:
{
"asi": true,
"undef": true
}

and this shell invocation:

jshint --config jshint.conf . | grep "is not defined" | grep -v require | grep -v console | grep -v exports | grep -v process | grep -v __dirname | grep -v module | grep -v assert | grep -v 'doc/'