querystring: remove eslint-disable

Remove the eslint-disable comments by using a strict comparison instead of a Boolean cast. PR-URL: nodejs#24995 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Сковорода Никита Андреевич <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Anto Aravinth <[email protected]>
refack · Jan 10, 2019 · cabdccb · cabdccb
1 parent aded0e2
commit cabdccb
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/lib/querystring.js b/lib/querystring.js
@@ -318,8 +318,7 @@ function parse(qs, sep, eq, options) {
               encodeCheck = 1;
               continue;
             } else if (encodeCheck > 0) {
-              // eslint-disable-next-line no-extra-boolean-cast
-              if (!!isHexTable[code]) {
+              if (isHexTable[code] === 1) {
                 if (++encodeCheck === 3)
                   keyEncoded = true;
                 continue;
@@ -348,8 +347,7 @@ function parse(qs, sep, eq, options) {
         if (code === 37/* % */) {
           encodeCheck = 1;
         } else if (encodeCheck > 0) {
-          // eslint-disable-next-line no-extra-boolean-cast
-          if (!!isHexTable[code]) {
+          if (isHexTable[code] === 1) {
             if (++encodeCheck === 3)
               valEncoded = true;
           } else {