jhnns/rewire — pull request #19 by mcandre (2013-05-09 16:14:42+00:00)
Title: Clearing away JSHint errors

I applied [JSHint](http://jshint.com/) to rewire to look for potential bugs. Originally, it printed 104 errors, but most of these were ignorable with `.jshintrc`. Which leaves us with:

$ jshint .
lib\bundlers\getRewireRegExp.js: line 12, col 12, Wrap the /regexp/ literal in parens to disambiguate the slash operator.

lib\bundlers\webpack\getLoaderTestRegExp.js: line 15, col 24, Missing semicolon.

lib\bundlers\webpack\webpackPostLoader.js: line 26, col 20, Possible strict violation.

lib\bundlers\webpack\webpackRewire.js: line 8, col 1, eval can be harmful.

lib\__get__.js: line 13, col 23, Bad assignment.
lib\__get__.js: line 15, col 16, eval can be harmful.

lib\__set__.js: line 15, col 23, Bad assignment.
lib\__set__.js: line 16, col 24, Bad assignment.
lib\__set__.js: line 17, col 19, Bad assignment.
lib\__set__.js: line 20, col 23, Bad assignment.
lib\__set__.js: line 24, col 48, Expected ';' and instead saw ')'.
lib\__set__.js: line 25, col 13, Expected an identifier and instead saw 'if' (a reserved word).
lib\__set__.js: line 25, col 16, Expected ':' and instead saw '('.
lib\__set__.js: line 25, col 64, Expected '}' to match '{' from line 24 and instead saw
')'.
lib\__set__.js: line 25, col 66, Expected ';' and instead saw '{'.
lib\__set__.js: line 26, col 36, Bad assignment.
lib\__set__.js: line 26, col 70, Expected ')' to match '(' from line 24 and instead saw
';'.
lib\__set__.js: line 27, col 31, Bad assignment.
lib\__set__.js: line 30, col 5, Expected '(end)' and instead saw '}'.

test\debug.test.js: line 7, col 9, Forgotten 'debugger' statement?

test\testModules\moduleA.js: line 35, col 47, 'navigator' is not defined.
test\testModules\moduleA.js: line 121, col 12, 'document' is not defined.

test\testModules\moduleB.js: line 35, col 47, 'navigator' is not defined.
test\testModules\moduleB.js: line 121, col 12, 'document' is not defined.

test\testModules\someOtherModule.js: line 3, col 1, Read only.

test\testModules\strictModule.js: line 4, col 34, Strict violation.

26 errors

I don't know enough about the rewire project to resolve these myself, just thought you should see.