From 69767ed23c8c5528c0909f2dda495a313f1f3ac7 Mon Sep 17 00:00:00 2001 From: Mike Pennisi Date: Sun, 27 Oct 2019 02:10:22 -0400 Subject: [PATCH] [[CHORE]] Align value with variable name The name `isLoneArrowParam` implies a value that is `true` when the identifier in question is a parameter for an arrow function. Update the value and the subsquent reference to align with this name. --- src/jshint.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/jshint.js b/src/jshint.js index 75f05e07bb..17224bc886 100644 --- a/src/jshint.js +++ b/src/jshint.js @@ -2247,11 +2247,11 @@ var JSHINT = (function() { // ...it should not be considered as a variable in the current scope. It // will be added to the scope of the new function when the next token is // parsed, so it can be safely ignored for now. - var isLoneArrowParam = state.tokens.next.id !== "=>"; + var isLoneArrowParam = state.tokens.next.id === "=>"; if (isReserved(context, this)) { warning("W024", this, v); - } else if (isLoneArrowParam && !state.funct["(comparray)"].check(v)) { + } else if (!isLoneArrowParam && !state.funct["(comparray)"].check(v)) { state.funct["(scope)"].block.use(v, state.tokens.curr); }