[[FIX]] Update parsing of object "rest" property

The latest version of the proposal disallows BindingPattern as the
right-hand side of a "rest" property.

tc39/proposal-object-rest-spread#50
http://tc39.github.io/tc39-notes/2017-05_may-23.html#16ih-why-allow-bindingpattern-for-bindingrestparameter-for-object-rest-maybe-we-should-just-allow-identifiers

Author: jugglinmike

src/jshint.js

@@ -3462,7 +3462,8 @@ var JSHINT = (function() {
       }
     };
     var assignmentProperty = function() {
-      var id;
+      var id, expr;
+
       if (checkPunctuator(state.tokens.next, "[")) {
         advance("[");
         expression(10);
@@ -3483,7 +3484,16 @@ var JSHINT = (function() {
             warning("W143", state.tokens.next, "object rest property", "objspreadrest");
           }
 
-          nextInnerDE();
+          // Due to visual symmetry with the array rest property (and the early
+          // design of the language feature), developers may mistakenly assume
+          // any expression is valid in this position.  Parse an expression and
+          // issue an error in order to recover more gracefully from this
+          // condition.
+          expr = expression(10);
+
+          if (expr.type !== "(identifier)") {
+            error("E030", expr, expr.value);
+          }
         } else {
           id = identifier();
         }