[[CHORE]] Revert parser improvements

JSHint's validation for assignment within conditional clauses is derived
from code first contributed by GitHub user "usrbincc." That user could
not be reached to sign JSHint's Contributor License Agreement. Revert
the user's contribution to the application logic (preserving the
corresponding MIT-licensed test code) so that it may be re-written under
the terms of the Contributor License Agreement.

Revert the changes introduced by
b3d054e

src/jshint.js

@@ -3171,9 +3171,11 @@ var JSHINT = (function() {
     if (!exprs.length) {
       return;
     }
+
+    exprs[exprs.length - 1].parent = true;
+
     if (exprs.length > 1) {
-      ret = Object.create(state.syntax[","]);
-      ret.exprs = exprs;
+      ret = Object.create(state.syntax[","], { exprs: { value: exprs } });
 
       first = exprs[0];
       last = exprs[exprs.length - 1];
@@ -3230,8 +3232,6 @@ var JSHINT = (function() {
       if (!isNecessary) {
         warning("W126", opening);
       }
-
-      ret.paren = true;
     }
 
     return ret;
@@ -3866,14 +3866,10 @@ var JSHINT = (function() {
   // For example: if (a = 1) { ... }
 
   function checkCondAssignment(expr) {
-    var id, paren;
-    if (expr) {
+    var id = expr.id;
+    if (id === ",") {
+      expr = expr.exprs[expr.exprs.length - 1];
       id = expr.id;
-      paren = expr.paren;
-      if (id === "," && (expr = expr.exprs[expr.exprs.length - 1])) {
-        id = expr.id;
-        paren = paren || expr.paren;
-      }
     }
     switch (id) {
     case "=":
@@ -3885,7 +3881,7 @@ var JSHINT = (function() {
     case "|=":
     case "^=":
     case "/=":
-      if (!paren && !state.option.boss) {
+      if (!expr.paren && !state.option.boss) {
         warning("W084");
       }
     }
@@ -5226,7 +5222,7 @@ var JSHINT = (function() {
     if (!state.option.asi)
       nolinebreak(this);
 
-    if (state.tokens.next.id !== ";" && !state.tokens.next.reach &&
+    if (state.tokens.next.id !== ";" &&
         state.tokens.curr.line === startLine(state.tokens.next)) {
       if (!state.funct["(scope)"].funct.hasLabel(v)) {
         warning("W090", state.tokens.next, v);
@@ -5254,7 +5250,7 @@ var JSHINT = (function() {
     if (!state.option.asi)
       nolinebreak(this);
 
-    if (state.tokens.next.id !== ";" && !state.tokens.next.reach) {
+    if (state.tokens.next.id !== ";") {
       if (state.tokens.curr.line === startLine(state.tokens.next)) {
         if (!state.funct["(scope)"].funct.hasLabel(v)) {
           warning("W090", state.tokens.next, v);