Re: [whatwg/streams] Fix invalid high water mark for tee branches (PR #1234)

@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