diff options
| author | cjihrig <cjihrig@gmail.com> | 2018-12-12 12:39:21 -0500 |
|---|---|---|
| committer | cjihrig <cjihrig@gmail.com> | 2018-12-14 13:13:43 -0500 |
| commit | 034ec648cad9dee4a67498aa674099ba1bc3c6ac (patch) | |
| tree | 204449150583938cec8666844ffe81e98d46c0a2 /lib/querystring.js | |
| parent | 37a5e01bda104eacca78da31afb9f9ec05da180c (diff) | |
| download | android-node-v8-034ec648cad9dee4a67498aa674099ba1bc3c6ac.tar.gz android-node-v8-034ec648cad9dee4a67498aa674099ba1bc3c6ac.tar.bz2 android-node-v8-034ec648cad9dee4a67498aa674099ba1bc3c6ac.zip | |
querystring: remove eslint-disable
Remove the eslint-disable comments by using a strict comparison
instead of a Boolean cast.
PR-URL: https://github.com/nodejs/node/pull/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>
Diffstat (limited to 'lib/querystring.js')
| -rw-r--r-- | lib/querystring.js | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/querystring.js b/lib/querystring.js index 5189a28f23..5839ca6b0d 100644 --- 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 { |