Fix for unnecessary semicolon + white

rauchg · Jul 27, 2012 · f0d5630 · f0d5630
1 parent 3a36171
commit f0d5630
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/jshint.js b/jshint.js
@@ -2149,6 +2149,9 @@ loop:   for (;;) {
         if (option.white) {
             left = left || token;
             right = right || nexttoken;
+            if (left.value === ";" && right.value === ";") {
+                return;
+            }
             if (left.line === right.line && left.character === right.from) {
                 left.from += (left.character - left.from);
                 warning("Missing space after '{a}'.",

diff --git a/tests/unit/options.js b/tests/unit/options.js
@@ -1161,3 +1161,20 @@ exports.browser = function () {
 	TestRun().test(src, { browser: true, undef: true });
 
 };
+
+exports.unnecessarysemicolon = function () {
+    var code = [
+        "function foo() {",
+        "    var a;;",
+        "}"
+    ];
+
+    TestRun()
+        .addError(2, "Unnecessary semicolon.")
+        .test(code);
+
+    TestRun()
+        .addError(2, "Unnecessary semicolon.")
+        .test(code, { white: true });
+
+};