diff options
author | Voltrex <mohammadkeyvanzade94@gmail.com> | 2021-08-04 00:25:38 +0430 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2021-08-12 15:49:53 -0700 |
commit | ae0cfc95d55f35b76be7341969d6ad36dcc000b7 (patch) | |
tree | d37f1ec8e16cb01fc5d8251c3aaac8a2e71de8a6 | |
parent | 0b69ac785174476505d7a0818b4206c0e86809a0 (diff) | |
download | node-new-ae0cfc95d55f35b76be7341969d6ad36dcc000b7.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); } |