Juriy Zaytsev
Juriy Zaytsev
Yeah, it could use some optimization. We can't return from forEach though.
Looks like JSHint issue. Plugged this in jshint.com and it showed "undefined" as unused.
Filed as https://github.com/jshint/jshint/issues/1604
:) @jdalton is the man!
Filed https://github.com/jshint/jshint/issues/1600 for 1st issue. Will look at others soon.
Fixed 2nd issue with https://github.com/kangax/jscritic/commit/15957a2e2a99607e328d945e9e898a845f76a22e
As for 3rd issue, it's the way JSHint detects so-called "implied" globals: ``` js if (typeof foo === 'undefined' && foo) { .. } ``` ^ here it catches `foo`,...
Yeah. Perhaps I should remove implieds from "global vars" entirely.
Filed this as https://github.com/jshint/jshint/issues/1605
Hmm, yeah I probably forgot to renew it. I don't think there's much value in this ancient project anymore? I wonder if removing CNAME would make https://kangax.github.io/jscritic work at least