Skip to content

Commit

Permalink
[[FIX]] Remove dead code for parameter parsing
Browse files Browse the repository at this point in the history
The removed code was made superfluous by a recent commit that unified
logic for function parameter parsing [1]. Since then, it has been
unreachable so it is safe to remove.

[1] Title: Re-use code to parse "fat arrow" function params
    Commit: 7430ee5
  • Loading branch information
jugglinmike committed Jan 14, 2015
1 parent 8e680e6 commit a1d5817
Showing 1 changed file with 2 additions and 14 deletions.
16 changes: 2 additions & 14 deletions src/jshint.js
Expand Up @@ -2452,7 +2452,6 @@ var JSHINT = (function() {
}, 155, true).exps = true;

prefix("(", function() {
var bracket, brackets = [];
var pn = state.tokens.next, pn1, i = -1;
var ret, triggerFnExpr;
var parens = 1;
Expand Down Expand Up @@ -2484,16 +2483,8 @@ var JSHINT = (function() {

if (state.tokens.next.id !== ")") {
for (;;) {
if (pn.value === "=>" && _.contains(["{", "["], state.tokens.next.value)) {
bracket = state.tokens.next;
bracket.left = destructuringExpression();
brackets.push(bracket);
for (var t in bracket.left) {
exprs.push(bracket.left[t].token);
}
} else {
exprs.push(expression(10));
}
exprs.push(expression(10));

if (state.tokens.next.id !== ",") {
break;
}
Expand All @@ -2509,9 +2500,6 @@ var JSHINT = (function() {
}
}

if (state.tokens.next.value === "=>") {
return exprs;
}
if (!exprs.length) {
return;
}
Expand Down

0 comments on commit a1d5817

Please sign in to comment.