Skip to content

Commit

Permalink
querystring: remove eslint-disable
Browse files Browse the repository at this point in the history
Remove the eslint-disable comments by using a strict comparison
instead of a Boolean cast.

PR-URL: #24995
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
cjihrig authored and BethGriggs committed Dec 18, 2018
1 parent bde5df2 commit 2e94f3b
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions lib/querystring.js
Original file line number Diff line number Diff line change
@@ -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 {

0 comments on commit 2e94f3b

Please sign in to comment.