diff options
author | Voltrex <mohammadkeyvanzade94@gmail.com> | 2021-08-04 00:25:38 +0430 |
---|---|---|
committer | Danielle Adams <adamzdanielle@gmail.com> | 2021-08-16 16:35:25 -0400 |
commit | ca3cb96d250b1a997d20e5f8c6dcc138b70d7dee (patch) | |
tree | bd40d0751487e6da1ff8ad02251e9e5ba5169d35 | |
parent | cc08d3062f9b0bace76eff382fe218e9198de66d (diff) | |
download | node-new-ca3cb96d250b1a997d20e5f8c6dcc138b70d7dee.tar.gz |
lib: cleanup validation
Used the `validateInteger()` validator to cleanup validation and keep
consistency.
PR-URL: https://github.com/nodejs/node/pull/39652
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
-rw-r--r-- | lib/internal/tty.js | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/lib/internal/tty.js b/lib/internal/tty.js index 5a247c7928..f8bfe8cf21 100644 --- a/lib/internal/tty.js +++ b/lib/internal/tty.js @@ -29,10 +29,7 @@ const { StringPrototypeToLowerCase, } = primordials; -const { - ERR_INVALID_ARG_TYPE, - ERR_OUT_OF_RANGE -} = require('internal/errors').codes; +const { validateInteger } = require('internal/validators'); let OSRelease; @@ -221,13 +218,9 @@ function hasColors(count, env) { env = count; count = 16; } else { - if (typeof count !== 'number') { - throw new ERR_INVALID_ARG_TYPE('count', 'number', count); - } - if (count < 2) { - throw new ERR_OUT_OF_RANGE('count', '>= 2', count); - } + validateInteger(count, 'count', 2); } + return count <= 2 ** getColorDepth(env); } |