Commit c38576e
buffer: simplify check size in assertSize
PR-URL: #18665
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 876e186 commit c38576e
1 file changed
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -237,9 +237,7 @@ function assertSize(size) { | |||
| 237 | 237 | ||
| 238 | 238 | if (typeof size !== 'number') { | |
| 239 | 239 | err = new errors.TypeError('ERR_INVALID_ARG_TYPE', 'size', 'number', size); | |
| 240 | - } else if (size < 0) { | ||
| 241 | - err = new errors.RangeError('ERR_INVALID_OPT_VALUE', 'size', size); | ||
| 242 | - } else if (size > kMaxLength) { | ||
| 240 | + } else if (size < 0 || size > kMaxLength) { | ||
| 243 | 241 | err = new errors.RangeError('ERR_INVALID_OPT_VALUE', 'size', size); | |
| 244 | 242 | } | |
| 245 | 243 | ||
0 commit comments