Removes a bad advise by JSHint to remove unnecessary semicolons.

Fixes GH-344

jshint.js

@@ -2343,11 +2343,8 @@ loop:   for (;;) {
     function statement(noindent) {
         var i = indent, r, s = scope, t = nexttoken;
 
-// We don't like the empty statement.
-
-        if (t.id === ';') {
-            warning("Unnecessary semicolon.", t);
-            advance(';');
+        if (t.id === ";") {
+            advance(";");
             return;
         }
 

tests/core.js

@@ -382,3 +382,20 @@ exports.testRawOnError = function () {
     assert.equal(JSHINT.errors[1].raw, 'Too many errors.');
     assert.equal(JSHINT.errors[2], null);
 };
+
+exports.yesEmptyStmt = function () {
+    var src = fs.readFileSync(__dirname + '/fixtures/emptystmt.js', 'utf8');
+
+    TestRun()
+        .addError(1, "Expected an identifier and instead saw ';'.")
+        .addError(6, "Expected an assignment or function call and instead saw an expression.")
+        .addError(10, "Unnecessary semicolon.")
+        .addError(17, "Unnecessary semicolon.")
+        .test(src, { curly: false });
+
+    TestRun()
+        .addError(1, "Expected an identifier and instead saw ';'.")
+        .addError(10, "Unnecessary semicolon.")
+        .addError(17, "Unnecessary semicolon.")
+        .test(src, { curly: false, expr: true });
+};

tests/fixtures/emptystmt.js

@@ -0,0 +1,23 @@
+var;
+
+var i = 0;
+while (++i < 10);
+
+foo;
+
+function foo() {
+    return;
+};
+
+if (true);
+
+for (var i = 0; i < 10; i += 1);
+
+for (var i = 0; i < 10; i += 1) {
+    foo();;
+    break;
+}
+
+for (var i = 0; i < 10; i += 1) {
+    continue;
+}