Re: [whatwg/streams] Refactor TransformStream's backpressure handling (#571)

ricea commented on this pull request.



>  
-  if (transformStream._readableBackpressure !== backpressure) {
-    TransformStreamSetBackpressure(transformStream, backpressure);
+  if (maybeBackpressure === true && transformStream._backpressure === false) {
+    // This allows pull() again. When desiredSize is 0, it's possible that a pull() will happen immediately (but
+    // asynchronously) after this because of pending read()s and set _backpressure back to false.
+    //
+    // We don't have to worry about that such a pull() may happen inside the enqueue() call, and therefore is not

How about:

> If pull() could be called from inside enqueue(), then this logic would be wrong. This cannot happen because there is always a promise pending from start() or pull() when _backpressure is false.

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