Re: [whatwg/streams] Update ReadableStreamTee to allow cloning for only the second branch (#528)

domenic commented on this pull request.



>        // There is no way to access the cloning code right now in the reference implementation.
       // If we add one then we'll need an implementation for StructuredClone.
-
+      // if (teeState.canceled2 === false && cloneForBranch2 === true) {
+      //   value2 = StructuredClone(value2);
+      // }

Ah, that is interesting; thanks for pointing it out. It is probably better this way, yeah.

-- 
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/528

Received on Tuesday, 11 October 2016 15:38:30 UTC