FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

stream: use validateBuffer for BYOB reader view · nodejs/node@374f7e2 · GitHub

/ node Public

Commit 374f7e2

Browse files
authored andcommitted
stream: use validateBuffer for BYOB reader view
The same check is already spelled validateBuffer(view, 'view') in ReadableStreamBYOBRequest.respondWithNewView(), and the thrown error is unchanged. Signed-off-by: greenhead <shren0812@gmail.com> PR-URL: #65046 Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent d7805e0 commit 374f7e2

1 file changed

Lines changed: 1 addition & 11 deletions

File tree

‎lib/internal/webstreams/readablestream.js‎

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1060,17 +1060,7 @@ class ReadableStreamBYOBReader {
10601060
async read(view, options = kEmptyObject) {
10611061
if (!isReadableStreamBYOBReader(this))
10621062
throw new ERR_INVALID_THIS('ReadableStreamBYOBReader');
1063-
if (!isArrayBufferView(view)) {
1064-
throw new ERR_INVALID_ARG_TYPE(
1065-
'view',
1066-
[
1067-
'Buffer',
1068-
'TypedArray',
1069-
'DataView',
1070-
],
1071-
view,
1072-
);
1073-
}
1063+
validateBuffer(view, 'view');
10741064
validateObject(options, 'options', kValidateObjectAllowObjectsAndNull);
10751065

10761066
const viewByteLength = ArrayBufferViewGetByteLength(view);

0 commit comments

Comments
 (0)

Back | FazBrowse Home | New Git URL