Re: [whatwg/streams] Add support for ReadableStream "owning" type (PR #1271)

@youennf commented on this pull request.



> @@ -2950,13 +2969,21 @@ The following abstract operations support the implementation of the
 <div algorithm>
  <dfn abstract-op lt="ReadableStreamDefaultControllerEnqueue"
  id="readable-stream-default-controller-enqueue">ReadableStreamDefaultControllerEnqueue(|controller|,
- |chunk|)</dfn> performs the following steps:
+ |chunk|, |transferList|)</dfn> performs the following steps:

I am not sure to follow, ReadableStreamDefaultControllerEnqueue should only be called within the streams spec.
Other specs should call https://streams.spec.whatwg.org/#readablestream-enqueue where the parameter is optional so this does not seem bad either way.
I can make it optional but, I tend to prefer being explicit there.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/1271#discussion_r1189554707
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/streams/pull/1271/review/1420076616@github.com>