- 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