- From: Yonathan Randolph <notifications@github.com>
- Date: Tue, 31 May 2022 20:05:29 -0700
- To: whatwg/streams <streams@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Wednesday, 1 June 2022 03:05:41 UTC
@yonran commented on this pull request.
> @@ -2073,7 +2073,7 @@ The following abstract operations operate on {{ReadableStream}} instances at a h
1. Perform ! [$InitializeReadableStream$](|stream|).
1. Let |controller| be a [=new=] {{ReadableByteStreamController}}.
1. Perform ? [$SetUpReadableByteStreamController$](|stream|, |controller|, |startAlgorithm|,
- |pullAlgorithm|, |cancelAlgorithm|, 0, undefined).
Thanks. I did not realize that [ReadableStreamDefaultControllerShouldCallPull](https://streams.spec.whatwg.org/#readable-stream-default-controller-should-call-pull) [ReadableByteStreamControllerShouldCallPull](https://streams.spec.whatwg.org/#readable-byte-stream-controller-should-call-pull) return early to so that when strategyHWM = 0, then no queuing is used. That is a very nice feature.
--
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/1234#discussion_r886280968
You are receiving this because you are subscribed to this thread.
Message ID: <whatwg/streams/pull/1234/review/991345472@github.com>
Received on Wednesday, 1 June 2022 03:05:41 UTC